prontolabs / pronto

Quick automated code review of your changes
MIT License
2.6k stars 245 forks source link

merge_base: revspec 'master' not found (Rugged::ReferenceError) #441

Open salujabhavesh opened 1 year ago

salujabhavesh commented 1 year ago

On running command :-

  1. pronto run: (error:- /home/rails/.rvm/gems/ruby-2.4.10/gems/pronto-0.10.0/lib/pronto/git/repository.rb:87:in `merge_base': revspec 'master' not found (Rugged::ReferenceError)

  2. bundle exec pronto run -c origin/master -f github_status -f github_pr_review --exit-code
    Getting error :- https://docs.github.com/rest/reference/pulls#list-pull-requests (Octokit::NotFound) + Faraday

Ruby Version :- 2.4.10 Rails Version :- 4.2.11 Linux version :- 16.04

gilesbowkett commented 3 months ago

I've seen this with Rugged in other contexts when the repo had main but not master, and I'm seeing it now for the same reason with Pronto. that makes it intuitive that main vs master would be the problem. but it seems like a really out-of-date problem, so I have doubts.

gilesbowkett commented 3 months ago

indeed, the main vs master aspect of this bug was first reported in 2016.

https://github.com/prontolabs/pronto/issues/191

per that page, using pronto run -c $(git name-rev --name-only HEAD) or pronto run -c main seems to fix it.