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

skip mend if PR is from a forked repo #120

Merged
merged 2 commits into from
Nov 6, 2024

Conversation

rahuls2997
Copy link
Contributor

@rahuls2997 rahuls2997 commented Oct 25, 2024

Summary

Added a check to skip mend workflow if PR is raised from a forked repo

Additional Context

When a PR was raised from a forked modules repo to main, the mend CI check is failing. This is because it requires mend token and the forked repo PR is unable to inherit the same. So to resolve the issue, we have added a step to check if the PR has been raised from a fork and if true, skip the PR pipeline.

Related Issues (if any)

Mention any related issues or pull requests.

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified.

@rahuls2997 rahuls2997 force-pushed the CAT-2117-skip-mend-for-PR-raised-from-forks branch from 9fb4f85 to 0452400 Compare October 28, 2024 08:46
Revert "test"

This reverts commit 6c43ddd.

resfs

e

test

test

t

test
@rahuls2997 rahuls2997 force-pushed the CAT-2117-skip-mend-for-PR-raised-from-forks branch from 2a2dd60 to 03e3359 Compare October 28, 2024 08:48
@rahuls2997 rahuls2997 changed the title skip mend if repo owner is not puppetlabs skip mend if PR is from a forked repo Oct 28, 2024
@rahuls2997 rahuls2997 marked this pull request as ready for review October 28, 2024 09:37
@rahuls2997 rahuls2997 requested a review from a team as a code owner October 28, 2024 09:37
Copy link
Contributor

@LukasAud LukasAud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@LukasAud LukasAud added the enhancement New feature or request label Nov 6, 2024
@david22swan david22swan merged commit fa22e46 into main Nov 6, 2024
2 checks passed
@david22swan david22swan deleted the CAT-2117-skip-mend-for-PR-raised-from-forks branch November 6, 2024 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Has been Actioned
Development

Successfully merging this pull request may close these issues.

3 participants