Skip to content

Added tox workflow for release tools (Infra)#885

Merged
fernando79513 merged 2 commits intomainfrom
add-tox-worflow-for-release-tools
Dec 11, 2023
Merged

Added tox workflow for release tools (Infra)#885
fernando79513 merged 2 commits intomainfrom
add-tox-worflow-for-release-tools

Conversation

@fernando79513
Copy link
Copy Markdown
Collaborator

@fernando79513 fernando79513 commented Dec 11, 2023

Description

We have added a new tox workflow for release tools, so we are running the tests anytime tools/release/ folder is modified.
The new workflow is called Test release tools with tox and will only be run using python 3.10.

Resolved issues

Closes CHECKBOX-1059

Documentation

Added a section of how to run the test under ./tools/release/README.md

Tests

In order to test the new tox workflow, go to ./tools/release/ and run tox -e py310.

The final behavior will be tested once the branch is merged and some new changes are done on the ./tools/release/ folder. In that case, the new github workflow should be run.

@fernando79513 fernando79513 requested a review from Hook25 December 11, 2023 13:37
@fernando79513 fernando79513 force-pushed the add-tox-worflow-for-release-tools branch from 0c7042f to 0af2f44 Compare December 11, 2023 13:58
 - Added tox yaml for github workflows
 - Included `tox.ini`, `.coveragerc` and `.gitignore`
@fernando79513 fernando79513 force-pushed the add-tox-worflow-for-release-tools branch from 0af2f44 to e4b5882 Compare December 11, 2023 14:00
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 11, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9f23c20) 35.70% compared to head (e4b5882) 36.06%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #885      +/-   ##
==========================================
+ Coverage   35.70%   36.06%   +0.35%     
==========================================
  Files         303      310       +7     
  Lines       34250    34603     +353     
  Branches     5917     5965      +48     
==========================================
+ Hits        12230    12478     +248     
- Misses      21458    21562     +104     
- Partials      562      563       +1     
Flag Coverage Δ
release-tools 70.25% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Copy Markdown
Collaborator

@Hook25 Hook25 left a comment

Choose a reason for hiding this comment

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

+1, ty for this

@fernando79513 fernando79513 merged commit ac632d0 into main Dec 11, 2023
@fernando79513 fernando79513 deleted the add-tox-worflow-for-release-tools branch December 11, 2023 14:45
kissiel pushed a commit that referenced this pull request Dec 13, 2023
* Added tox workflow for release tools

 - Added tox yaml for github workflows
 - Included `tox.ini`, `.coveragerc` and `.gitignore`

* Added how to test to documentation
kissiel pushed a commit that referenced this pull request Dec 13, 2023
* Added tox workflow for release tools

 - Added tox yaml for github workflows
 - Included `tox.ini`, `.coveragerc` and `.gitignore`

* Added how to test to documentation
binli pushed a commit to binli/checkbox that referenced this pull request Mar 22, 2024
* Added tox workflow for release tools

 - Added tox yaml for github workflows
 - Included `tox.ini`, `.coveragerc` and `.gitignore`

* Added how to test to documentation
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