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

Issue 58: Add new GitHub Actions #62

Merged
merged 14 commits into from
Feb 21, 2024
Merged

Issue 58: Add new GitHub Actions #62

merged 14 commits into from
Feb 21, 2024

Conversation

seabbs
Copy link
Collaborator

@seabbs seabbs commented Feb 21, 2024

This PR closes #58 by adding a range of GitHub Actions. Note that until all actions are passing it is not likely worht reviewing.

Copy link
Collaborator Author

@seabbs seabbs left a comment

Choose a reason for hiding this comment

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

Note it also moves us to use the sciml style for JuliaFormatter and runs this everywhere.

Copy link
Collaborator Author

@seabbs seabbs left a comment

Choose a reason for hiding this comment

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

Note it also moves us to use the sciml style for JuliaFormatter and runs this everywhere.

@SamuelBrand1
Copy link
Collaborator

LGTM!

@seabbs
Copy link
Collaborator Author

seabbs commented Feb 21, 2024

Note I am not 100% that coverage will work properly because of the workarounds I've had to put in place to get this all working in a monorepo.

This might be starting to indicate that we want to move to a multi-repo structure but we can discuss that down the line I think

@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered ☂️

@seabbs seabbs merged commit d672c2c into main Feb 21, 2024
5 checks passed
@seabbs seabbs deleted the issue58 branch February 21, 2024 21: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.

Setup suggest Julia actions
3 participants