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

[Critical Bug] Error when updating an existing PR #175

Open
dukeofcool199 opened this issue Jul 20, 2023 · 4 comments
Open

[Critical Bug] Error when updating an existing PR #175

dukeofcool199 opened this issue Jul 20, 2023 · 4 comments

Comments

@dukeofcool199
Copy link

i get the following error when trying to run spr diff on an existing pr

🔁 Commit was changed - updating Pull Request #378
🛑 GitHub: Validation Failed
Documentation URL: https://docs.github.com/rest/pulls/pulls#update-a-pull-request
Errors:
- {"code":"invalid","field":"base","message":"Proposed base branch 'refs/heads/spr/dukeofcool199/
is
invalid","resource":"PullRequest"}

@dukeofcool199 dukeofcool199 changed the title Error when updating an existing PR [Critical Bug] Error when updating an existing PR Jul 21, 2023
@indigoviolet
Copy link

@sven-of-cord I just got a very similar error while trying to spr land:

❯ spr land
dc3afed Minor bugfixes for Doc and Engine.instantiate_doc
  #️⃣   Pull Request #18
  🛫  Getting started...
  🛑  GitHub: Validation Failed
      Documentation URL: https://docs.github.com/rest/pulls/pulls#update-a-pull-request
      Errors:
      - {"code":"invalid","field":"base","message":"Proposed base branch 'refs/heads/main' is invalid","resource":"PullRequest"}

@eko-wibowo
Copy link

I just got this for spr land not clear why it says invalid base :-/ any workaround?

❯ spr land
05b36a4 [SI-97] Integrate new scheduler in chat
  #️⃣   Pull Request #5489
  🛫  Getting started...
  🛑  GitHub: Validation Failed
      Documentation URL: https://docs.github.com/rest/pulls/pulls#update-a-pull-request
      Errors:
      - {"code":"invalid","field":"base","message":"Proposed base branch 'refs/heads/main' is invalid","resource":"PullRequest"}

@SlowSnowFox
Copy link

a temporary fix is to go onto GitHub and change the base branch manually to be main. Only do this after you landed the dependent PR's. Then spr land works.

@SlowSnowFox
Copy link

building from source completely removes this problem. So removing the latest version from homebrew(1.3.4) and building from scratch (1.3.5-beta.1) is the way to go.

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

No branches or pull requests

4 participants