Skip to content

fix: dependabot PRs were not running CI after merge #374

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

Merged
merged 1 commit into from
Jul 15, 2025

Conversation

fmenezes
Copy link
Collaborator

@fmenezes fmenezes commented Jul 15, 2025

Proposed changes

fix dependabot PRs not running CI after merge

Checklist

@fmenezes fmenezes marked this pull request as ready for review July 15, 2025 13:28
@Copilot Copilot AI review requested due to automatic review settings July 15, 2025 13:28
@fmenezes fmenezes requested a review from a team as a code owner July 15, 2025 13:28
Copy link
Contributor

@Copilot 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

This PR introduces a dedicated workflow to auto-merge Dependabot pull requests using a bot token and cleans up the legacy merge step from the fork-specific code health workflow.

  • Add a new workflow to auto-merge Dependabot PRs on main with an app token
  • Remove the outdated merge-dependabot-pr job from the code_health_fork.yaml

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/dependabot_pr.yaml New workflow to detect Dependabot PRs and enable auto-merge via bot
.github/workflows/code_health_fork.yaml Remove redundant Dependabot merge job from the fork code health CI
Comments suppressed due to low confidence (2)

.github/workflows/dependabot_pr.yaml:5

  • [nitpick] The workflow only triggers on pull_request opened events, so if Dependabot updates the PR later, auto-merge won't be re-enabled. Consider adding 'synchronize' and 'reopened' to the event types.
    types: [opened]

.github/workflows/dependabot_pr.yaml:9

  • An empty permissions block removes all default access. Explicitly specify only the permissions you need (e.g., contents: read) to avoid unexpected failures and improve clarity.
permissions: {}

@fmenezes fmenezes enabled auto-merge (squash) July 15, 2025 13:31
@fmenezes fmenezes merged commit 48bce63 into main Jul 15, 2025
20 checks passed
@fmenezes fmenezes deleted the fix_dependabot_pr_ci branch July 15, 2025 13:34
@coveralls
Copy link
Collaborator

Pull Request Test Coverage Report for Build 16294620032

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.005%) to 77.39%

Totals Coverage Status
Change from base Build 16292877152: 0.005%
Covered Lines: 2828
Relevant Lines: 3611

💛 - Coveralls

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.

4 participants