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.

Sign up for GitHub

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

Jump to bottom

semgrep script: specify repository during checkout step #2943

Merged
merged 1 commit into from
Jul 18, 2023
Merged

semgrep script: specify repository during checkout step #2943

merged 1 commit into from
Jul 18, 2023

Conversation

onkarvhanumante
Copy link
Contributor

This is needed when PR is raised from fork branch. In this case repository will be fork repo and correct version of code will be checkout

Testing:

  • test onkarvhanumante/prebid-server#6
    PR raised on onkarvhanumante/prebid-server from onkarhanumante30/prebid-server repo. This mimics scenario where adapter forks raise PR against main prebid-server repo 
          fetch-depth: 0
      ref: patch-2
      repository: onkarhanumante30/prebid-server

specify repository during checkout step
1f15ff1 
This is needed when PR is raised from fork branch. In this case repository will be fork repo
@onkarvhanumante onkarvhanumante changed the title semgrep script: specify repository during checkout step semgrep script: specify repository during checkout step Jul 18, 2023
@Sonali-More-Xandr Sonali-More-Xandr self-assigned this Jul 18, 2023
@gargcreation1992 gargcreation1992 self-assigned this Jul 18, 2023
@Sonali-More-Xandr Sonali-More-Xandr merged commit 6c89af2 into master Jul 18, 2023
3 checks passed
@cross-repo-issue-creation cross-repo-issue-creation bot mentioned this pull request Jul 18, 2023
Closed
@onkarvhanumante onkarvhanumante deleted the semgrepPRs/specify-repo branch July 18, 2023 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gargcreation1992 gargcreation1992 gargcreation1992 approved these changes

@Sonali-More-Xandr Sonali-More-Xandr Sonali-More-Xandr approved these changes

Assignees

@gargcreation1992 gargcreation1992

@Sonali-More-Xandr Sonali-More-Xandr

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@onkarvhanumante @gargcreation1992 @Sonali-More-Xandr