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

CI: limit the on push triggers #38291

Merged
merged 2 commits into from
Mar 22, 2023
Merged

Conversation

XhmikosR
Copy link
Member

No description provided.

@XhmikosR XhmikosR added the CI label Mar 21, 2023
@XhmikosR
Copy link
Member Author

I'll give this another go.

@XhmikosR XhmikosR marked this pull request as ready for review March 22, 2023 07:03
@XhmikosR XhmikosR merged commit c2671ec into main Mar 22, 2023
@XhmikosR XhmikosR deleted the revert-37657-revert-35574-main-xmr-ci branch March 22, 2023 07:03
@XhmikosR XhmikosR added the skip-changelog So that the release drafter action doesn't include it label Mar 22, 2023
@XhmikosR
Copy link
Member Author

@GeoSot @julien-deramond so this works quite well minus the BrowserStack workflow. Right now it only runs when something is merged in main.

I'm tempted to revert this again... What we basically want is the BrowserStack tests to run on all of our branches, just not on PRs from other people.

@XhmikosR
Copy link
Member Author

XhmikosR commented Mar 22, 2023

OK, I added fdcb9b1 and this seems to restore the previous behavior. If anyone notices any other issues please let me know.

@mahilanmjd mahilanmjd mentioned this pull request Apr 16, 2023
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI skip-changelog So that the release drafter action doesn't include it
Projects
No open projects
Status: No status
Development

Successfully merging this pull request may close these issues.

1 participant