Skip to content
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

🎨 Retry push attempts #57

Merged
merged 1 commit into from
Oct 26, 2022
Merged

🎨 Retry push attempts #57

merged 1 commit into from
Oct 26, 2022

Conversation

scthi
Copy link
Contributor

@scthi scthi commented Oct 26, 2022

Type of Change

  • Bugfix
  • Enhancement / new feature
  • Refactoring
  • Documentation

Description

Even with pull before push in place, rejections can occur. With the retry, this chance should be minimized.

Current branch main is up to date.
To https://github.com/Staffbase/mops.git
 ! [rejected]        main -> main (fetch first)
error: failed to push some refs to 'https://github.com/Staffbase/***.git'
hint: Updates were rejected because the remote contains work that you do
hint: not have locally. This is usually caused by another repository pushing
hint: to the same ref. You may want to first integrate the remote changes
hint: (e.g., 'git pull ...') before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.
Error: Process completed with exit code 1.

Checklist

  • Write tests
  • Make sure all tests pass
  • Update documentation
  • Review the Contributing Guideline and sign CLA
  • Reference relevant issue(s) and close them after merging

@github-actions
Copy link

github-actions bot commented Oct 26, 2022

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

Even with pull before push in place, rejections can occur.
With the retry, this chance should be minimized.

I have read the CLA Document and I hereby sign the CLA
@scthi scthi force-pushed the retry-push-to-mops branch from 3099677 to 199fa2d Compare October 26, 2022 10:52
@scthi
Copy link
Contributor Author

scthi commented Oct 26, 2022

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Oct 26, 2022
@scthi scthi marked this pull request as ready for review October 26, 2022 10:59
@scthi scthi requested a review from a team as a code owner October 26, 2022 10:59
@scthi scthi requested review from monotek and ricoberger October 26, 2022 10:59
Copy link

@seomin seomin left a comment

Choose a reason for hiding this comment

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

LGTM

@scthi scthi merged commit 910ddf0 into main Oct 26, 2022
@scthi scthi deleted the retry-push-to-mops branch October 26, 2022 13:12
@github-actions github-actions bot locked and limited conversation to collaborators Oct 26, 2022
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.

3 participants