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

Always run lint, regardless of branch #1492

Merged
merged 1 commit into from
May 27, 2024
Merged

Conversation

jonesmz
Copy link
Contributor

@jonesmz jonesmz commented May 21, 2024

Someone working on a branch in their own fork won't see the lint running on their github-actions CI until they open a pull request.

It'd be much easier to ensure changes being worked on are going to pass the linter if the linter runs before the last step of opening the PR.

@eakraly eakraly merged commit 525550a into coturn:master May 27, 2024
44 of 54 checks passed
@jonesmz jonesmz deleted the always-run-lint branch May 27, 2024 06:16
maddymeows pushed a commit to maddymeows/coturn that referenced this pull request Jun 2, 2024
Someone working on a branch in their own fork won't see the lint running
on their github-actions CI until they open a pull request.

It'd be much easier to ensure changes being worked on are going to pass
the linter if the linter runs before the last step of opening the PR.
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.

None yet

2 participants