Skip to content

Conversation

@B-a-S
Copy link
Contributor

@B-a-S B-a-S commented Aug 27, 2023

Prevent runs NVIDIA_CI on the forks.

@janjust
Copy link
Contributor

janjust commented Aug 28, 2023

@B-a-S Does this need to be cherry-picked to other branches?
v5.0.x?

Copy link
Member

@jsquyres jsquyres left a comment

Choose a reason for hiding this comment

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

This seems to solve the immediate problem, but it doesn't fix the issue for people who make PR's on forks (which isn't nearly as common as pushing to a fork, but it does happen).

I'll approve this PR so that people stop getting false failures on their forks when they simply push branches, but can you please also investigate more / better ways to make this github action only run on the main Open MPI repo?

@jsquyres jsquyres merged commit f4965c4 into open-mpi:main Aug 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants