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 CI testing for PRs #14

Merged
merged 1 commit into from
Jun 5, 2024
Merged

Add CI testing for PRs #14

merged 1 commit into from
Jun 5, 2024

Conversation

rtamalin
Copy link
Collaborator

@rtamalin rtamalin commented Jun 5, 2024

Add a GitHub actions workflow to run our tests when a PR is proposed or updated.

Update Makefile's .PHONY list to include all of our targets.

Update README.md with instructions on how to run tests and update some stale text.

Closes: #13

Add a GitHub actions workflow to run our tests when a PR is proposed
or updated.

Update Makefile's .PHONY list to include all of our targets.

Update README.md with instructions on how to run tests and update some
stale text.

Closes: #13
@rtamalin rtamalin requested a review from mbelur June 5, 2024 12:35
Copy link

@brett060102 brett060102 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rtamalin rtamalin merged commit 430e607 into main Jun 5, 2024
1 check passed
@rtamalin rtamalin deleted the add_basic_ci_testing branch July 24, 2024 13:20
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.

Add PR CI testing infrastructure
2 participants