WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@caugner
Copy link
Contributor

@caugner caugner commented Oct 29, 2025

Description

Updates all on: pull_request workflows to run on all PRs, not just on PRs targeting main.

Motivation

The condition is unnecessary, and might obfuscate test failures.

Additional details

Related issues and pull requests

Part of mdn/fred#1020.

Not only PRs targeting `main`.
@caugner caugner requested a review from a team as a code owner October 29, 2025 16:17
@caugner caugner requested review from argl and removed request for a team October 29, 2025 16:17
@github-actions github-actions bot added system [PR only] Infrastructure and configuration for the project size/s [PR only] 6-50 LoC changed labels Oct 29, 2025
@argl argl merged commit c83df62 into main Oct 30, 2025
19 checks passed
@argl argl deleted the check-all-prs branch October 30, 2025 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/s [PR only] 6-50 LoC changed system [PR only] Infrastructure and configuration for the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants