-
Notifications
You must be signed in to change notification settings - Fork 1k
Enable Dependabot for packages directory with weekly grouped updates #11250
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
Conversation
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
create-cloudflare
@cloudflare/kv-asset-handler
miniflare
@cloudflare/pages-shared
@cloudflare/unenv-preset
@cloudflare/vite-plugin
@cloudflare/vitest-pool-workers
@cloudflare/workers-editor-shared
@cloudflare/workers-utils
wrangler
commit: |
- Remove invalid changeset that included private packages - Change directory from '/packages' to '/' for proper workspace support - Add open-pull-requests-limit: 1 to ensure single PR per week Co-Authored-By: [email protected] <[email protected]>
.github/dependabot.yml
Outdated
| allow: | ||
| - dependency-name: "workerd" | ||
| - dependency-name: "@cloudflare/workers-types" | ||
| # Check for dependency updates in packages directory |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this comment is now out of date
- Add ignore filters for workerd and @cloudflare/workers-types to the weekly dependency update entry - Update comment to clarify this entry excludes workerd & workers-types - This resolves the 'overlapping directories' error by making the two npm entries non-overlapping Co-Authored-By: [email protected] <[email protected]>
Dependabot does not allow two npm update entries with the same directory and target-branch, regardless of allow/ignore filters. The uniqueness constraint only considers (package-ecosystem, directory, target-branch). Removing the weekly entry to resolve the overlapping directories error. Will discuss alternative approaches with maintainers. Co-Authored-By: [email protected] <[email protected]>
Devin PR requested by [email protected]
This PR configures Dependabot to automatically check for dependency updates in the packages directory, grouping all updates into a single PR that runs weekly on Sundays at 6am.
Changes
Added a new Dependabot configuration entry to
.github/dependabot.ymlthat:/packagesdirectory for npm dependency updatespackages-dependenciesgroupincreaseversioning strategydependenciesandskip-pr-description-validationlabelsReview Checklist
Important: Please verify the following:
/packagesdirectory path is correct for Dependabot. Dependabot typically scans directories containingpackage.jsonfiles. Since/packagesis a parent directory with multiple subdirectories, this configuration may need adjustment to work as intended.*will correctly group all package dependencies into a single PR as requested.Link to Devin run: https://app.devin.ai/sessions/9af3cd534c0940efbdbd7a6f160d8a23