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

Disable test report upload when origin is a fork #1198

Closed
commjoen opened this issue Jan 22, 2024 · 2 comments
Closed

Disable test report upload when origin is a fork #1198

commjoen opened this issue Jan 22, 2024 · 2 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@commjoen
Copy link
Collaborator

We see https://github.com/OWASP/wrongsecrets/blob/master/.github/workflows/main.yml#L68 step failing which should publish the test report in a PR when the origin of the PR is a fork of the OWASP/Wrongsecrets repo.

This issue is about disabling the step in the action if the origin of the PR is a fork.

@commjoen commjoen added bug Something isn't working help wanted Extra attention is needed labels Jan 22, 2024
@commjoen commjoen reopened this Jan 28, 2024
@commjoen
Copy link
Collaborator Author

64b2905#diff-7829468e86c1cc5d5133195b5cb48e1ff6c75e3e9203777f6b2e379d9e4882b3

if: ${{ github.event.pull_request.head.repo.full_name != 'OWASP/wrongsecrets' }} && (success() || failure())
        if: ${{ github.event.pull_request.head.repo.full_name == 'OWASP/wrongsecrets' }} && (success() || failure())

both did not work...

commjoen added a commit that referenced this issue Jan 28, 2024
@commjoen
Copy link
Collaborator Author

#1223 did not fix it either?

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

No branches or pull requests

1 participant