Skip to content

Conversation

@GeekMasher
Copy link
Contributor

This pull request introduces several updates to the Config struct and its implementation in src/config.rs, as well as modifications to the defaults.yml file. The key changes include adding new fields for ecosystem and global excludes, and ensuring these fields are properly handled during configuration loading and processing.

Changes to Config struct and implementation:

  • src/config.rs: Added ecosystem and excludes fields to the Config struct and updated the Default implementation to initialize these fields. [1] [2]
  • src/config.rs: Updated the Config implementation to handle the new ecosystem field by pushing its value to the ecosystems vector if present.
  • src/config.rs: Added logic to handle the new excludes field by extending the excludes list for each location in the locations vector if excludes is not empty.

Updates to default configuration:

  • defaults.yml: Added a new entry for "CodeQL Packs" with associated paths, excludes, ecosystems, and patterns.

@GeekMasher GeekMasher requested a review from Copilot January 8, 2025 11:10
@github-actions
Copy link

github-actions bot commented Jan 8, 2025

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

Tip: If you use Visual Studio Code, you can request a review from Copilot before you push from the "Source Control" tab. Learn more

@GeekMasher GeekMasher enabled auto-merge (rebase) January 8, 2025 11:12
@GeekMasher GeekMasher merged commit 9f47e79 into main Jan 8, 2025
6 checks passed
@GeekMasher GeekMasher deleted the config-eco-excludes branch January 8, 2025 11:14
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.

2 participants