Skip to content

Do not fail diff informed analyses when analyze is run twice in the same job - #2876

Merged
henrymercer merged 4 commits into
mainfrom
henrymercer/fix-diff-informed-multiple-analyze
May 1, 2025
Merged

Do not fail diff informed analyses when analyze is run twice in the same job#2876
henrymercer merged 4 commits into
mainfrom
henrymercer/fix-diff-informed-multiple-analyze

Conversation

@henrymercer

@henrymercer henrymercer commented May 1, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Tolerate the diff range extension pack directory already existing and overwrite it, to avoid failing in jobs that call analyze more than once.

Also defer the computation of the dependency caching default cache config to avoid getting the value of the RUNNER_TEMP environment variable before it can be overridden by test logic.

And finally disable diff-informed analyses within the config parsing tests, since these tests do not account for the additional query filter added by diff-informed analyses.

Merge / deployment checklist

  • Confirm this change is backwards compatible with existing workflows.
  • Confirm the readme has been updated if necessary.
  • Confirm the changelog has been updated if necessary.