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

Run github CI actions on fork PRs #3020

Merged
merged 1 commit into from
Feb 4, 2025

Conversation

jesszzzz
Copy link
Contributor

@jesszzzz jesszzzz commented Feb 4, 2025

Motivation

CI tests should run on PRs from forked repos

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

CI is running on this PR

Related Issues and PRs

(Is this PR part of a group of changes? Link the other relevant PRs and Issues here. Use https://help.github.com/en/articles/closing-issues-using-keywords for help on GitHub syntax)

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 4, 2025
@jesszzzz jesszzzz marked this pull request as ready for review February 4, 2025 15:36
@jesszzzz jesszzzz mentioned this pull request Feb 4, 2025
Copy link

@tonykao8080 tonykao8080 left a comment

Choose a reason for hiding this comment

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

looks good

@jesszzzz jesszzzz merged commit e824695 into facebookresearch:main Feb 4, 2025
24 checks passed
@jesszzzz jesszzzz deleted the run_ci_on_fork_pr branch February 4, 2025 17:49
michaelriedl pushed a commit to michaelriedl/hydra that referenced this pull request Feb 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants