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

Add per-mixin workflow for linting #1348

Merged
merged 5 commits into from
Oct 22, 2024
Merged

Add per-mixin workflow for linting #1348

merged 5 commits into from
Oct 22, 2024

Conversation

Dasomeone
Copy link
Member

@Dasomeone Dasomeone commented Oct 22, 2024

This PR reintroduces mixin linting via the use of mixtool, but in an updated fashion to only account for changes made to any given mixins in the current working push request.

This is done to avoid blocking PR authors when mixins outwith the changes they're making fail linting.

@Dasomeone Dasomeone force-pushed the dasomeone/update_ci branch from b86a7f7 to 24bbda0 Compare October 22, 2024 12:40
@Dasomeone Dasomeone requested a review from a team as a code owner October 22, 2024 12:40
@Dasomeone Dasomeone force-pushed the dasomeone/update_ci branch 5 times, most recently from 72a67ae to 914d5df Compare October 22, 2024 12:58
@Dasomeone Dasomeone force-pushed the dasomeone/update_ci branch from 1c71456 to 72e5225 Compare October 22, 2024 13:21
@Dasomeone Dasomeone changed the title [WIP] Add per-mixin workflow for linting Add per-mixin workflow for linting Oct 22, 2024
@Dasomeone Dasomeone enabled auto-merge (squash) October 22, 2024 14:44
@Dasomeone Dasomeone merged commit 866f8a7 into master Oct 22, 2024
4 checks passed
@Dasomeone Dasomeone deleted the dasomeone/update_ci branch October 22, 2024 14:44
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.

2 participants