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

Comment on alert only if the PR is made from main repo #250

Merged
merged 2 commits into from
Oct 3, 2024

Conversation

OptimumCode
Copy link
Collaborator

Adds condition to add PR comment only if it is made from the main repo.
Also uses a GitHub token as a fallback in case the secret is not available

Resolves #249.

I made the PR from fork to check how it works

@OptimumCode OptimumCode requested review from krzema12 and aSemy October 1, 2024 18:52
@OptimumCode OptimumCode marked this pull request as ready for review October 1, 2024 19:26
@OptimumCode
Copy link
Collaborator Author

Hi, @aSemy. Could you please take a look at the PR if you have time (@krzema12 is on vacation at the moment)?

Copy link
Owner

@krzema12 krzema12 left a comment

Choose a reason for hiding this comment

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

Thanks! 🙇

@krzema12 krzema12 merged commit 29a51f2 into krzema12:main Oct 3, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Collecting benchmark results in PR is broken
2 participants