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

Syncing through forked repo hits "shallow update not allowed" after the first pull request is merged into target repo #270

Open
GangWang01 opened this issue Nov 11, 2022 · 0 comments
Labels
bug Something isn't working

Comments

@GangWang01
Copy link

🐞 Describe the bug

I synced with FORK parameter referring to Fork and pull request workflow.
The action could successfully create the first pull request targeting to the target repo through forked repo. Once the first pull request is merged, when I push some change to the sync file next time in source repo the action run failed due to shallow update not allowed error, like https://github.com/GangWang01/Experiment/actions/runs/3443890910.
The error is a result of main branch of forked repo is some commits behind target repo. To work around this problem, I have to manually sync forked repo from its upstream(the target repo). However it doesn't make sense for the purpose of this action.

BTW, ORIGINAL_MESSAGE and COMMIT_AS_PR_TITLE don't work when syncing file through forked repo.

📚 To reproduce

  1. https://github.com/GangWang01/Experiment is the source repo and it's configured to use this action with FORK parameter. This is my own repo I have write access.
  2. https://github.com/JaynieBai/ManageTestResults is the target repo. I don't have write access to it.
  3. Merge the first pull request created by the action.
  4. Push some change to the file that needs to sync to source repo. Then check if the run fails with the error.

💡 Expected behavior

It's very helpful syncing through forked repo could work well.

🖼️ Screenshots

No response

⚙️ Environment

Neither BetaHuhn/repo-file-sync-action@v1 nor BetaHuhn/repo-file-sync-action@latest works.

📋 Additional context

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant