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

ci: fixups of analysis workflow #4572

Merged
merged 2 commits into from
Dec 27, 2024
Merged

Conversation

lgritz
Copy link
Collaborator

@lgritz lgritz commented Dec 26, 2024

  • Fix missing secrets that accidentally disabled the analysis workflow a few commits back. After splitting into reusable steps, we needed the main analysis workflow to let the steps workflow inherit secrets.
  • Run analysis on TOT dev branches.
  • For analysis, only build OIIO as debug, not dependencies.

* Fix missing secrets that accidentally disabled the analysis workflow
  a few commits back. After splitting into reusable steps, we needed
  the main analysis workflow to let the steps workflow inherit secrets.
* Run analysis on TOT dev branches.
* For analysis, only build OIIO as debug, not dependencies.

Signed-off-by: Larry Gritz <[email protected]>
@lgritz
Copy link
Collaborator Author

lgritz commented Dec 27, 2024

Pre-emptively merging since this only affects CI, irrefutably fixes something that is broken, and doesn't change code or user build scripts.

@lgritz lgritz merged commit af252a1 into AcademySoftwareFoundation:main Dec 27, 2024
30 checks passed
lgritz added a commit to lgritz/OpenImageIO that referenced this pull request Dec 27, 2024
* Fix missing secrets that accidentally disabled the analysis workflow a
few commits back. After splitting into reusable steps, we needed the
main analysis workflow to let the steps workflow inherit secrets.
* Run analysis on TOT dev branches.
* For analysis, only build OIIO as debug, not dependencies.

---------

Signed-off-by: Larry Gritz <[email protected]>
@lgritz lgritz deleted the lg-sonar branch December 27, 2024 22:43
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.

1 participant