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

refactor(ci): split general workflow into testing and linting #17

Merged
merged 3 commits into from
Jan 1, 2024

Conversation

gierens
Copy link
Member

@gierens gierens commented Nov 1, 2023

This splits the general CI workflow into linting and testing, and also adds specific paths for triggering the CI. This prevents unnecessary tests when only the README is changed for example.

@apognu
Copy link
Contributor

apognu commented Nov 13, 2023

I was watching the repo to verify my tests were working and found this. You probably already know, but the mocked tests are failing because the installation lib nss_wrapper got booted during the split.

@PThorpe92 PThorpe92 merged commit aa6f0f8 into main Jan 1, 2024
@PThorpe92 PThorpe92 deleted the linting-workflow branch January 1, 2024 15:17
@gierens
Copy link
Member Author

gierens commented Feb 1, 2024

@PThorpe92 awesome, thanks for fixing this, you're the best ❤️

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.

3 participants