Skip to content

feat: Add git pull support - #373

Merged
ad-m merged 1 commit into
masterfrom
add-pull-support
May 23, 2026
Merged

ad-m merged 1 commit into
masterfrom
add-pull-support

Conversation

@ZPascal

@ZPascal ZPascal commented May 20, 2026 •

Copy link
Copy Markdown
Collaborator

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

Test

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

Copilot AI left a comment

Copy link
Copy Markdown

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

ZPascal commented May 23, 2026 •

Copy link
Copy Markdown
Collaborator Author

@ad-m or @jackton1, can you please review the PR?

@ad-m

ad-m commented May 23, 2026

Copy link
Copy Markdown
Owner

Looks fine to me

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