Skip to content

Conversation

AbhishekBhaskar
Copy link
Contributor

This PR adds the exclude-paths option in dependabot-cli job config. This options allows the user to specify paths to manifest files in sub-directories which should be ignored during Dependabot updates.
Related epic - dependabot/dependabot-core#4364

@AbhishekBhaskar AbhishekBhaskar requested review from a team as code owners August 6, 2025 20:31
@AbhishekBhaskar AbhishekBhaskar self-assigned this Aug 6, 2025
@honeyankit honeyankit self-requested a review August 6, 2025 20:36
@honeyankit
Copy link
Contributor

@AbhishekBhaskar It is failing few test cases

@kbukum1 kbukum1 self-requested a review August 7, 2025 20:17
@kbukum1
Copy link
Contributor

kbukum1 commented Aug 7, 2025

@AbhishekBhaskar It is failing few test cases

@honeyankit , these tests are already failing on smoke test. It is related to npm, swift package updates that are not ignored. Because they are in transitive dependency and ignore conditions aren't capable to ignore new updates for transitive dependencies.

@kbukum1 kbukum1 requested a review from Copilot August 7, 2025 20:19
Copy link

@Copilot 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.

Pull Request Overview

This PR adds support for the exclude-paths attribute to the Dependabot CLI job configuration, allowing users to specify manifest file paths in subdirectories that should be ignored during dependency updates.

  • Adds ExcludePaths field to the Job struct with appropriate JSON and YAML tags
  • Includes test case demonstrating the new exclude-paths configuration with example paths

Reviewed Changes

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

File Description
internal/model/job.go Adds ExcludePaths field to Job struct with JSON/YAML serialization tags
internal/model/job_test.go Adds test configuration showing exclude-paths usage with example directory paths

@AbhishekBhaskar AbhishekBhaskar added this pull request to the merge queue Aug 7, 2025
Merged via the queue into main with commit a4c1378 Aug 7, 2025
80 of 84 checks passed
@AbhishekBhaskar AbhishekBhaskar deleted the abhishekbhaskar/add-exclude-paths-attr branch August 7, 2025 20:37
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.

3 participants