-
Notifications
You must be signed in to change notification settings - Fork 1k
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
fix(github): allow Github App to clone forks of a private repository #2044
Conversation
9552950
to
1d2317b
Compare
fix: only fetch "pull/$PR_NUMBER/head" when githubAppEnabled
1d2317b
to
832f666
Compare
@nishkrishnan @chenrui333 @jamengual could i get some feedback? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add some tests for this? Thanks!
@joshuasimon-taulia looks pretty straightforward, can you please add some tests? It should be good to go after that. 🙏 |
are there any existing test cases i can easily copy/paste from? i took a peek into https://github.com/runatlantis/atlantis/blob/7a927c050b945ccae64850f327c1309c2448aa2a/server/events/github_app_working_dir_test.go, but i don’t see an immediately obvious way to mock a private github fork. |
Alright, that is fine then. |
thanks for the contribution @joshuasimon-taulia |
…ntis#2044) fix: only fetch "pull/$PR_NUMBER/head" when githubAppEnabled
closes #1696
i baked an image with my changes. running atlantis against private forks now works in my installation