V2 #59
Merged
V2 #59
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This commit introduces pytest code coverage for better insights into test execution. This is accomplished through the use of 'coverage' and 'pytest-cov' tools. Moreover, Sphinx documentation generation and the GitHub Pages deployment steps were removed from the workflow.
Added a title, a centered WrenchCL logo, some relevant badges, and a short description about the WrenchCL library to the documentation home page. This should provide visitors a quick overview of the project.
In this commit, the 'test_tools.py' file has been refactored to enhance readability by separating import statements. Moreover, the CI/CD workflow on GitHub Actions has been improved to handle pull requests on branch 'v2'. It now runs tests in
Kydoimos97
had a problem deploying
to
pypi-deployment
GitHub Actions
Failure
Added lines of code in the GitHub workflow .yml file to create a 'coverage' directory and move the lcov.info file into this directory before uploading the coverage report. This will help organize the generated test coverage reports better.
Kydoimos97
had a problem deploying
to
pypi-deployment
GitHub Actions
Failure
The pytest command in the GitHub workflow file, "publish-to-pypi.yml", has been simplified. Extra reports generation like xml and html has been removed, focusing only on lcov coverage reports. Additionally, the creation of the coverage directory and the move of lcov.info file operations have been removed, as they're not necessary.
Kydoimos97
temporarily deployed
to
pypi-deployment
GitHub Actions
Inactive
The commit eliminates the step of posting a coverage report in the GitHub Actions workflow for publishing to Pypi. This was done to streamline the workflow and avoid clutter of coverage reports in pull request comments.
Kydoimos97
temporarily deployed
to
pypi-deployment
GitHub Actions
Inactive
A link to the project documentation on Read the Docs was added to the README file. This will make it easier for users to get detailed information and instructions about the project.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.