Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create matrix for fuzzing #457

Merged
merged 1 commit into from May 9, 2024
Merged

Create matrix for fuzzing #457

merged 1 commit into from May 9, 2024

Conversation

Alan-Jowett
Copy link
Collaborator

This pull request includes changes to the .github/workflows/fuzzing.yml file, which is part of the GitHub Actions workflow. The changes primarily focus on replacing the use of environment variables with a matrix strategy for the job configurations. This allows for more flexibility in defining and managing the job configurations.

Here are the most important changes:

  • Replaced the env: block with a strategy: block that uses a matrix to define platform and arch parameters. This change allows for more flexibility in defining and managing the job configurations.

  • Updated the runs-on: field to use the matrix.platform value instead of the env.platform value. This change is in line with the switch to a matrix strategy.

  • Updated the run: field in the "Generate the cache key" step to use the matrix.platform and matrix.arch values instead of the env.platform and env.arch values. This change ensures that the cache key is generated correctly based on the matrix values.

  • Updated the if: conditions in various steps to use the matrix.platform value instead of the env.platform value. This change ensures that the correct steps are executed based on the platform defined in the matrix. [1] [2]

  • Removed the scan_build environment variable and the corresponding conditional logic. This simplifies the workflow as the scan_build variable was not being used elsewhere.

  • Changed the DCMAKE_BUILD_TYPE value from a variable (env.build_type) to a fixed string ("Debug"). This change simplifies the workflow and ensures that the build type is always "Debug" for this job.

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>
@Alan-Jowett Alan-Jowett merged commit 45aefaa into main May 9, 2024
10 checks passed
@Alan-Jowett Alan-Jowett deleted the fix_ci_cd branch May 9, 2024 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant