Skip to content

Fix error introduced by merging branch without updating first#125

Merged
3mcd merged 1 commit into
mainfrom
kalilsn/merge-issue
Oct 5, 2023
Merged

Fix error introduced by merging branch without updating first#125
3mcd merged 1 commit into
mainfrom
kalilsn/merge-issue

Conversation

@kalilsn
Copy link
Copy Markdown
Contributor

@kalilsn kalilsn commented Oct 5, 2023

I refactored this code in my branch, someone else added a new endpoint in a different branch and merged that change, then I merged my branch. This is the scenario that the Require branches to be up to date before merging branch protection rule helps with, but we don't have that since this is an unpaid private repo.

For now, I've settled for a reminder option
image

@isTravis isTravis requested a deployment to kalilsn/merge-issue - core PR #125 October 5, 2023 15:13 — with Render Abandoned
@isTravis isTravis temporarily deployed to kalilsn/merge-issue - core PR #125 October 5, 2023 15:13 — with Render Destroyed
@isTravis isTravis temporarily deployed to kalilsn/merge-issue - integration-evaluations PR #125 October 5, 2023 15:14 — with Render Destroyed
@isTravis isTravis temporarily deployed to kalilsn/merge-issue - integration-submissions PR #125 October 5, 2023 15:14 — with Render Destroyed
@3mcd 3mcd merged commit 1afc48d into main Oct 5, 2023
@3mcd 3mcd deleted the kalilsn/merge-issue branch October 5, 2023 15:18
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.

3 participants