Skip to content
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

Fix build.gradle files missing some updates #6624

Merged
merged 1 commit into from
Feb 9, 2023

Conversation

deivid-rodriguez
Copy link
Contributor

When multiple configurations include the same dependency, only one would be updated.

Fixes #6598.

@deivid-rodriguez deivid-rodriguez requested a review from a team as a code owner February 8, 2023 13:20

raise "Expected content to change!" if updated_content == buildfile.content

updated_file(file: buildfile, content: updated_content)
end

def original_buildfile_declaration(dependency, requirement)
def original_buildfile_declarations(dependency, requirement)
# This implementation is limited to declarations that appear on a
# single line.
Copy link
Member

Choose a reason for hiding this comment

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

A reminder that we'd benefit greatly from:

When multiple configurations include the same dependency, only one would
be updated.
@jeffwidman jeffwidman force-pushed the deivid-rodriguez/dependabot-core-6598 branch from d03794f to 6517e50 Compare February 9, 2023 23:02
@jeffwidman jeffwidman merged commit 74c4607 into main Feb 9, 2023
@jeffwidman jeffwidman deleted the deivid-rodriguez/dependabot-core-6598 branch February 9, 2023 23:12
alcere pushed a commit that referenced this pull request Feb 20, 2023
Within a single file, when multiple configurations include the same dependency, only one would
be updated.
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.

[BUG] dependabot-core does not update all dependencies in build.gradle
2 participants