Keep user-provided query filters first by cklin · Pull Request #2938 · github/codeql-action (original) (raw)

@cklin

Using defaultAugmentationProperties to generated expected test output eliminates the need to change individual tests when we add a new field to AugmentationProperties.

@cklin marked this pull request as ready for review

June 19, 2025 19:04

Copilot AI review requested due to automatic review settings

June 19, 2025 19:04

@cklin

This commit renames AugmentationProperties.defaultQueryFilters to extraQueryExclusions, and changes its type to ExcludeQueryFilter[]. It matters whether we are adding query inclusions or exclusions, and this renaming makes that distinction clearer.

@cklin

@cklin

aeisenberg

@cklin cklin deleted the cklin/default-query-filters branch

June 19, 2025 20:46

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})