Skip to content

Comments

Docstring coverage CI/CD (51)#83

Closed
JMP-MO wants to merge 5 commits intoACCESS-Community-Hub:developfrom
JMP-MO:Add-interrogate-package-51
Closed

Docstring coverage CI/CD (51)#83
JMP-MO wants to merge 5 commits intoACCESS-Community-Hub:developfrom
JMP-MO:Add-interrogate-package-51

Conversation

@JMP-MO
Copy link
Collaborator

@JMP-MO JMP-MO commented Apr 2, 2025

I have created a new workflow to install the interrogate package.
Run the package using a github workflow.
Output the following:

  • Detailed report text file.
  • Badge to display.

Currently an existing task (black) is failing in the pre-commit workflow. Which I am unsure of its origin, but am sure this failed for previous push requests so may need looking into if this is just me or failing for others. This workflow completes successfully.

Screenshot 2025-04-02 at 16 06 01

@JMP-MO
Copy link
Collaborator Author

JMP-MO commented Apr 2, 2025

I have set the interrogate pass rate to 40% so the test passes. However we may want to increase this pass rate at a later point.

@JMP-MO JMP-MO changed the title Add interrogate package 51 Docstring coverage CI/CD (51) Apr 2, 2025
@JMP-MO JMP-MO closed this Apr 2, 2025
@coveralls
Copy link

Pull Request Test Coverage Report for Build 14223036869

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 57.568%

Totals Coverage Status
Change from base Build 14196779914: 0.0%
Covered Lines: 7716
Relevant Lines: 13003

💛 - Coveralls

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