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.

Sign up for GitHub

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

Jump to bottom

🧪 Fix & properly enforce lint.yml #38

Merged

kammce merged 1 commit into main from expand-lint

Sep 2, 2024

Member

kammce commented Sep 1, 2024 •

edited

Loading

Add include/ directory to lint
Fix regex to skip third_party directories in lint

kammce force-pushed the expand-lint branch 2 times, most recently from 363ae11 to 20a211b Compare

September 2, 2024 01:24


          🧪 Fix & properly enforce lint.yml

452e107

- Add `include/` directory to lint
- Fix regex to skip third_party directories in lint

kammce force-pushed the expand-lint branch from 20a211b to 452e107 Compare

September 2, 2024 01:39

kammce changed the title ~~👷 Add lint to includes~~ 🧪 Fix & properly enforce lint.yml

Member Author

kammce commented Sep 2, 2024

The failing tests here are what we are expecting.

kammce merged commit d6f4619 into main

39 of 44 checks passed

kammce deleted the expand-lint branch

September 2, 2024 01:53

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet