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

Run unit tests on PRs #62

Merged
merged 1 commit into from
Feb 22, 2024
Merged

Run unit tests on PRs #62

merged 1 commit into from
Feb 22, 2024

Conversation

swgillespie
Copy link
Collaborator

@swgillespie swgillespie commented Feb 22, 2024

#56 broke unit tests but we didn't know because we don't try to run tests until after the linter runs, and the linter was having problems.

@swgillespie swgillespie requested a review from a team as a code owner February 22, 2024 04:38
@swgillespie swgillespie merged commit 3adf198 into main Feb 22, 2024
5 checks passed
@swgillespie swgillespie deleted the swgillespie/run-tests-3 branch February 22, 2024 04:47
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

1 participant