Skip to content

mergify: add mergify configuration#114

Merged
nathan-weinberg merged 2 commits into
instructlab:mainfrom
nathan-weinberg:mergify
Aug 28, 2024
Merged

mergify: add mergify configuration#114
nathan-weinberg merged 2 commits into
instructlab:mainfrom
nathan-weinberg:mergify

Conversation

@nathan-weinberg

Copy link
Copy Markdown
Contributor

This PR does the following:

  • Adds the following Mergify automation
    • Auto-merging PRs with 2 approving reviews and successful CI
    • Auto-labeling for PRs that touch certain files
  • Updates lint action
    • Hardens several steps
    • Aligns moreso to the CLI repo

Signed-off-by: Nathan Weinberg <nweinber@redhat.com>
@nathan-weinberg
nathan-weinberg requested a review from a team August 28, 2024 14:04
@nathan-weinberg nathan-weinberg linked an issue Aug 28, 2024 that may be closed by this pull request

@bjhargrave bjhargrave left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Otherwise it looks fine.

Comment thread .github/mergify.yml Outdated
Comment thread .github/mergify.yml Outdated
Comment thread .github/mergify.yml
Signed-off-by: Nathan Weinberg <nweinber@redhat.com>
Comment thread .github/workflows/lint.yml

jobs:
spellcheck:
name: Spellcheck (en_US)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

could you explain this deletion?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@nathan-weinberg
nathan-weinberg merged commit eaa977d into instructlab:main Aug 28, 2024
@nathan-weinberg
nathan-weinberg deleted the mergify branch August 28, 2024 16:14
@mergify mergify Bot added the CI/CD Affects CI/CD configuration label Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD Affects CI/CD configuration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Introuduce merge automation via Mergify

5 participants