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 coverage report generation to GitHub workflows #162

Merged
merged 5 commits into from
Feb 10, 2024
Merged

Conversation

SmetDenis
Copy link
Owner

A new workflow has been added into the main.yml to generate test coverage reports. It runs on the latest Ubuntu and uses actions for checkout, Java setup, Clojure tools installation, and caching Leiningen dependencies. Moreover, the 'lein-cloverage' plugin was added to the project.clj replacing the 'jonase/eastwood' plugin.

Denis Smet added 5 commits February 10, 2024 22:25
A new workflow has been added into the main.yml to generate test coverage reports. It runs on the latest Ubuntu and uses actions for checkout, Java setup, Clojure tools installation, and caching Leiningen dependencies. Moreover, the 'lein-cloverage' plugin was added to the project.clj replacing the 'jonase/eastwood' plugin.
The test case that runs pi-iter 5000 times has been removed in the file ex_1_31_test.clj. This removal was necessary because running pi-iter 5000 is impossible in Clojure at the moment due to system limitations. The other test cases remain unchanged.
Coverage report generation switched from codecov to lcov format. In the Coveralls config, the lcov path was updated and the 'continue-on-error' attribute was set to true to prevent workflow halting on errors. Removed reliance on parallel flag and run number as they are no longer required.
Added a Coveralls badge to display test coverage status in the README file. This enhancement ensures that the code coverage information is visible and directly accessible from the repository's main page. Also, migrated from codecov to lcov for coverage report generation.
The caching mechanism for Leiningen dependencies has been removed in this commit. The relevant lines in the GitHub actions workflow had been deleted, this was done with the intention to streamline the workflow by eliminating unnecessary cache steps.
@SmetDenis SmetDenis merged commit 3fe2e5f into main Feb 10, 2024
6 checks passed
@SmetDenis SmetDenis deleted the cloverage branch February 10, 2024 18:53
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.

1 participant