Skip to content

Come home, dependabot#5290

Merged
Swiddis merged 1 commit intoopensearch-project:mainfrom
Swiddis:infra/come-home-dependabot
Mar 31, 2026
Merged

Come home, dependabot#5290
Swiddis merged 1 commit intoopensearch-project:mainfrom
Swiddis:infra/come-home-dependabot

Conversation

@Swiddis
Copy link
Copy Markdown
Collaborator

@Swiddis Swiddis commented Mar 30, 2026

Description

Adds a quick YML config to poke it back to life. We can remove the config again once it's alive if we want to keep the default config.

Some decisions:

  • Weekly updates since daily tends to flood a lot, and our stale PR review is biweekly anyway
  • The group config means all the updates come in one PR, instead of dozens of PRs for every update
  • Skip changelog by default

Related Issues

N/A

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • New functionality has javadoc added.
  • New functionality has a user manual doc added.
  • New PPL command checklist all confirmed.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff or -s.
  • Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Simeon Widdis <sawiddis@amazon.com>
@github-actions
Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🧪 No relevant tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ Recommended focus areas for review

Grouping Config

The group configuration syntax appears incorrect. The correct Dependabot v2 syntax for grouping dependencies uses groups (plural) as the key, not group. This may cause the grouping feature to be silently ignored, resulting in individual PRs per dependency update rather than a single grouped PR.

group:
  all-dependencies:
    patterns:
      - "*"

Copy link
Copy Markdown
Collaborator

@dai-chen dai-chen left a comment

Choose a reason for hiding this comment

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

Do you know why other repo like sql-cli doesn't need this?

@Swiddis
Copy link
Copy Markdown
Collaborator Author

Swiddis commented Mar 31, 2026

Not sure the exact dependabot behavior, but I know you can configure it with this & if it's reconfigured it'll come back. There might be another way to revive it without a file bump but even then I'd rather push this config if we're enabling it to prevent as much PR load.

@Swiddis Swiddis merged commit 51d874b into opensearch-project:main Mar 31, 2026
41 of 42 checks passed
@Swiddis Swiddis deleted the infra/come-home-dependabot branch March 31, 2026 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infrastructure Changes to infrastructure, testing, CI/CD, pipelines, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants