Skip to content

ci: group sdk updates#4651

Open
andresmr wants to merge 1 commit intomainfrom
dependabot/group-sdk-updates
Open

ci: group sdk updates#4651
andresmr wants to merge 1 commit intomainfrom
dependabot/group-sdk-updates

Conversation

@andresmr
Copy link
Collaborator

Description

Group SDK updates

Copilot AI review requested due to automatic review settings February 12, 2026 10:08
Copy link
Contributor

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

Updates Dependabot configuration to better control Gradle dependency update behavior and group selected SDK updates into a single PR stream targeting develop.

Changes:

  • Adds broader Gradle ignore rules plus additional explicitly ignored DHIS2-related dependencies.
  • Introduces a Dependabot group intended to group org.hisp.dhis:android-core updates together (patch/minor/major).
  • Keeps weekly GitHub Actions updates with unchanged PR limit.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +18 to +23
# Allow security updates for all dependencies
- dependency-name: "*"
update-types: ["version-update:semver-major", "version-update:semver-minor"]
# But allow security updates even for major/minor
except:
- update-types: ["version-update:semver:security"]
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

The ignore entry adds an except: block, but except is not a valid key in Dependabot’s ignore configuration. Additionally, version-update:semver:security is not a valid update-types value. This will likely cause Dependabot to reject the config. If the goal is to ensure security updates still open, note that Dependabot security updates are handled separately and generally aren’t affected by ignore; remove except and keep only supported update-types entries (or restructure using a separate update config).

Suggested change
# Allow security updates for all dependencies
- dependency-name: "*"
update-types: ["version-update:semver-major", "version-update:semver-minor"]
# But allow security updates even for major/minor
except:
- update-types: ["version-update:semver:security"]
# Ignore non-security major/minor updates for all dependencies.
# Dependabot security updates are handled separately and will still be opened.
- dependency-name: "*"
update-types: ["version-update:semver-major", "version-update:semver-minor"]

Copilot uses AI. Check for mistakes.
Comment on lines 33 to +40
groups:
gradle-updates:
dhis2-android-core:
patterns:
- "*" # Group all Gradle updates into one PR
- "org.hisp.dhis:android-core"
update-types:
- "patch" # Only include patch updates (no major or minor versions)
- "patch"
- "minor"
- "major"
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

The global ignore rule for dependency-name: "*" ignoring semver-major/minor will also apply to org.hisp.dhis:android-core, so the dhis2-android-core group’s minor/major update-types won’t ever be proposed. If you want to allow minor/major updates only for android-core while keeping the rest patch-only, consider adding a second updates: entry for Gradle that allows just org.hisp.dhis:android-core (with its group settings), and keep the global ignore in the other entry.

Copilot uses AI. Check for mistakes.
@sonarqubecloud
Copy link

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.

1 participant