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

Setting up various badges for the repository #2

Merged
merged 26 commits into from
Jun 24, 2024

Conversation

cmacmackin
Copy link
Collaborator

This commit will add badges for CI status, code coverage, etc. to the repository README. It will also update the CI system a bit. It is still a work-in-progress.

@makeclean
Copy link

This is all good stuff, but we should be moving stuff to use INL's CI system CIVET which is much more integrated with MOOSE development and when developemen tis further along will be ran by INL to make sure they don' t break app

@cmacmackin
Copy link
Collaborator Author

That should probably be added to the "Backlog" column for the project.

Copy link

codecov bot commented Jun 11, 2024

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

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

@cmacmackin
Copy link
Collaborator Author

In the task for this PR it says to include badges for "Dependency (for MFEM)". What is meant by this? The version we depend on? At present I think we're still depending on a particular commit of a fork, so that doesn't really apply.

@cmacmackin
Copy link
Collaborator Author

I've added all of the badges I can think of at the moment. I often like to put in a badge for the status of building/deploying documentation, but we can think about that once some has been written.

Copy link
Contributor

@alexanderianblair alexanderianblair left a comment

Choose a reason for hiding this comment

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

Thanks Chris - this looks good, and should help with reporting code quality of new commits going forward.

Happy for this to go in as-is; minor comment to trim the cclake build script is optional and could be implemented in a later PR.

scripts/build-platypus-csd3-cclake.sh Show resolved Hide resolved
@alexanderianblair alexanderianblair merged commit dc141df into main Jun 24, 2024
14 checks passed
@alexanderianblair alexanderianblair deleted the cmacmackin/badges branch July 14, 2024 12:46
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