Skip to content
This repository was archived by the owner on Feb 11, 2026. It is now read-only.

fix: Mergify and Dependabot configs YAML are incorrect/inconsistent#772

Merged
vishnoianil merged 1 commit into
mainfrom
fix-dependabot-and-mergify-configs
May 2, 2025
Merged

fix: Mergify and Dependabot configs YAML are incorrect/inconsistent#772
vishnoianil merged 1 commit into
mainfrom
fix-dependabot-and-mergify-configs

Conversation

@courtneypacheco
Copy link
Copy Markdown
Contributor

@courtneypacheco courtneypacheco commented May 2, 2025

Overview

Some of the YAML configurations are inconsistent and/or incorrect (and are not being actively parsed). Example invalid config for Dependabot:

Screenshot 2025-05-02 at 12 42 43 PM

Therefore, the Dependabot updates we want are not properly being triggered.

Proposed Changes

  • Fixed the Dependabot config
  • Updated the mergify config to have uniform formatting
  • Added a new lint check to lint YAML files in the .github directory. (Other directories are out of scope for this PR, but we can lint and fix other broken YAML files in a follow-up PR.)
  • Added a yamllint config so we can configure how we want yamllint to behave in the above lint check.

Some of the YAML configurations are inconsistent and/or incorrect (and are not being actively parsed).

To ensure these files continue to be parsed, I have also introduced a YAML linter for the `.github` directory. There might be a bit of work to lint other YAML files. That is out of scope for this PR, though.

Signed-off-by: Courtney Pacheco <6019922+courtneypacheco@users.noreply.github.com>
@courtneypacheco courtneypacheco force-pushed the fix-dependabot-and-mergify-configs branch from 69c65c9 to b5cff91 Compare May 2, 2025 16:38
@courtneypacheco courtneypacheco requested review from a team, Gregory-Pereira, aevo98765, chungtan, juancappi, n1hility and renrenstart and removed request for a team May 2, 2025 16:47
Copy link
Copy Markdown
Contributor

@vishnoianil vishnoianil left a comment

Choose a reason for hiding this comment

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

LGTM

@vishnoianil
Copy link
Copy Markdown
Contributor

@courtneypacheco This looks great!. Thanks for the PR!. Really appreciate it.

@vishnoianil vishnoianil merged commit 0e396ad into main May 2, 2025
9 checks passed
@ktdreyer ktdreyer deleted the fix-dependabot-and-mergify-configs branch May 2, 2025 17:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants