Skip to content

Conversation

@acuteaangle
Copy link
Contributor

This adds GitHub Actions integration to automatically build (and run tests, should they be added in the future) on Windows, MacOS, and Linux.

This will (hopefully) prevent issues like gh-87 (introduced by gh-83) from recurring in the future.

Related-to: gh-83, gh-86, gh-87

@acuteaangle acuteaangle marked this pull request as ready for review January 4, 2025 09:19
@prochac
Copy link
Contributor

prochac commented Jan 4, 2025

Does it pass if it's run on top of #86 ? Then it should, right?

@acuteaangle
Copy link
Contributor Author

It does, here are the logs and results.

This adds GitHub Actions integration to automatically build (and run
tests, should they be added in the future) on Windows, MacOS, and Linux.

This will (hopefully) prevent issues like Spagett1gh-87 (introduced by Spagett1gh-83)
from recurring in the future.

Related-to: Spagett1gh-83, Spagett1gh-86, Spagett1gh-87
@acuteaangle
Copy link
Contributor Author

force-pushed to resolve trailing whitespace and correct linux job name to test.

1:  0989ea8fc4ac ! 1:  cd95a0db0afb ci: add github ci intergration to ensure PRs build successfully
@@ .github/workflows/tests.yaml (new)
 +  - push
 +  - pull_request
 +
-+env: 
++env:
 +  CARGO_TERM_COLOR: always
 +
 +jobs:
-+  linux:
++  test:
 +    name: Build and Test
 +    runs-on: ${{ matrix.os }}
 +    strategy:

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