Skip to content

Conversation

@fleroux514
Copy link
Contributor

@fleroux514 fleroux514 commented Jun 30, 2023

Relates to #399

Complement to fix made in #401

Fixing the use case when manually merging a PR with the following digger.yml config:

  • auto-merge: true
  • on_commit_to_default: [digger apply]

Digger would run the apply cmd and then verify if the PR is mergeable (for the auto-merge config), which would result in a failure.

@fleroux514 fleroux514 requested review from Spartakovic and veziak June 30, 2023 01:44
Co-authored-by: Mohamed Habib <moe.habib9@gmail.com>
@fleroux514 fleroux514 requested a review from motatoes June 30, 2023 15:08
@motatoes motatoes merged commit 81a192d into develop Jun 30, 2023
ben-of-codecraft pushed a commit to ben-of-codecraft/digger that referenced this pull request May 21, 2024
…q#422)

* fix: verify if manually merged with auto-merge config

---------

Co-authored-by: Mohamed Habib <moe.habib9@gmail.com>
breardon2011 pushed a commit that referenced this pull request Sep 17, 2025
* fix: verify if manually merged with auto-merge config

---------

Co-authored-by: Mohamed Habib <moe.habib9@gmail.com>
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.

2 participants