Skip to content

Remove the dependabot workaround#37526

Merged
AndriySvyryd merged 3 commits intomainfrom
copilot/merge-dependabot-config
Jan 18, 2026
Merged

Remove the dependabot workaround#37526
AndriySvyryd merged 3 commits intomainfrom
copilot/merge-dependabot-config

Conversation

Copy link
Contributor

Copilot AI commented Jan 16, 2026

Consolidates dependabot-managed package versions into the root Directory.Packages.props and removes the separate eng/dependabot directory workaround.

Changes

  • Directory.Packages.props: Merged package versions (Azure SDK, SQL Server, external deps) from eng/dependabot/Directory.Packages.props and removed the import
  • dependabot.yml: Updated nuget directory from /eng/dependabot to /
  • Deleted: eng/dependabot/Directory.Packages.props and eng/dependabot/global.json
Original prompt

Merge eng/dependabot
/Directory.Packages.props into ./Directory.Packages.props, remove eng/dependabot and update the dependabot config accordingly


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…pendabot config

Co-authored-by: AndriySvyryd <6539701+AndriySvyryd@users.noreply.github.com>
Copilot AI changed the title [WIP] Merge dependabot config and remove eng/dependabot Merge eng/dependabot into root Directory.Packages.props Jan 16, 2026
Copilot AI requested a review from AndriySvyryd January 16, 2026 20:27
@AndriySvyryd AndriySvyryd changed the title Merge eng/dependabot into root Directory.Packages.props Remove the dependabot workaround Jan 16, 2026
@AndriySvyryd AndriySvyryd marked this pull request as ready for review January 16, 2026 20:52
@AndriySvyryd AndriySvyryd requested a review from a team as a code owner January 16, 2026 20:53
Copilot AI review requested due to automatic review settings January 16, 2026 20:53
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.

Pull request overview

This PR consolidates dependabot-managed package versions into the root Directory.Packages.props file and removes the workaround directory that was created due to a dependabot limitation with global.json handling.

Changes:

  • Merged 15 external dependency package versions from eng/dependabot/Directory.Packages.props into the root Directory.Packages.props
  • Updated dependabot configuration to monitor the root directory instead of eng/dependabot
  • Removed the eng/dependabot directory and its contents (Directory.Packages.props and global.json)

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
Directory.Packages.props Added 15 package versions (Azure SDK, SQL Server, and external dependencies) and removed the import of the dependabot directory
.github/dependabot.yml Updated nuget directory from /eng/dependabot to /
eng/dependabot/Directory.Packages.props Deleted - contents merged into root Directory.Packages.props
eng/dependabot/global.json Deleted - empty file used as workaround

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Member

@roji roji left a comment

Choose a reason for hiding this comment

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

🐑 🇮🇹

@AndriySvyryd AndriySvyryd merged commit af8a90c into main Jan 18, 2026
10 checks passed
@AndriySvyryd AndriySvyryd deleted the copilot/merge-dependabot-config branch January 18, 2026 17:39
Copy link
Member

@ViktorHofer ViktorHofer left a comment

Choose a reason for hiding this comment

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

How does dependabot now know which dependencies to update and which not? (i.e. Arcade.Sdk should never get updated)

@AndriySvyryd
Copy link
Member

@ViktorHofer I'll add it to the ignore list if it tries to update it

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.

5 participants