Skip to content

feat: Add git pull support#373

Open
ZPascal wants to merge 1 commit into
masterfrom
add-pull-support
Open

feat: Add git pull support#373
ZPascal wants to merge 1 commit into
masterfrom
add-pull-support

Conversation

@ZPascal
Copy link
Copy Markdown
Collaborator

@ZPascal ZPascal commented May 20, 2026

Add pull input — pull before push support

Summary

Adds an optional pull input that performs a git pull before the push step.
This prevents push failures caused by diverged histories, especially in workflows
where multiple jobs or runners commit to the same branch concurrently.

fix: #84

TODO:

  • Test the functionality

@ZPascal ZPascal self-assigned this May 20, 2026
@ZPascal ZPascal requested a review from Copilot May 20, 2026 21:05
Copy link
Copy Markdown

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

Adds an optional pull input to the GitHub Push Action so it can integrate upstream changes (git pull) right before pushing, reducing push failures when multiple runners/jobs update the same branch.

Changes:

  • Introduces pull input with selectable strategies (rebase, merge, ff-only, true).
  • Updates start.sh to run a pull step before the existing push step.
  • Documents the new input and provides a workflow example in the README.

Reviewed changes

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

File Description
start.sh Adds conditional pre-push pull logic with strategy selection and remote URL construction.
README.md Documents the new pull input and adds an example workflow showing usage.
action.yml Exposes the new pull input (default false) in the action metadata.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread start.sh Outdated
Comment thread start.sh Outdated
Comment thread action.yml Outdated
@ZPascal ZPascal force-pushed the add-pull-support branch from ffc6238 to cbe6715 Compare May 21, 2026 04:37
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