Skip to content

Commit

Permalink
Fix readme badges and expected number of tests (#1235)
Browse files Browse the repository at this point in the history
Some stuff that was missing from #1213 

- Updates readme badges to point to the renamed gh actions
- Tells codecov to expect 14 batches of coverage data, not 10 (we used
to have 6 regression + 4 unit, now 6 regression + 8 unit)
  • Loading branch information
f0uriest authored Aug 29, 2024
2 parents 416b5fb + 788a197 commit 2ebdf6a
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
8 changes: 4 additions & 4 deletions README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -111,12 +111,12 @@ Contribute
:target: https://desc-docs.readthedocs.io/en/latest/?badge=latest
:alt: Documentation

.. |UnitTests| image:: https://github.com/PlasmaControl/DESC/actions/workflows/unittest.yml/badge.svg
:target: https://github.com/PlasmaControl/DESC/actions/workflows/unittest.yml
.. |UnitTests| image:: https://github.com/PlasmaControl/DESC/actions/workflows/unit_tests.yml/badge.svg
:target: https://github.com/PlasmaControl/DESC/actions/workflows/unit_tests.yml
:alt: UnitTests

.. |RegressionTests| image:: https://github.com/PlasmaControl/DESC/actions/workflows/regression_test.yml/badge.svg
:target: https://github.com/PlasmaControl/DESC/actions/workflows/regression_test.yml
.. |RegressionTests| image:: https://github.com/PlasmaControl/DESC/actions/workflows/regression_tests.yml/badge.svg
:target: https://github.com/PlasmaControl/DESC/actions/workflows/regression_tests.yml
:alt: RegressionTests

.. |Codecov| image:: https://codecov.io/gh/PlasmaControl/DESC/branch/master/graph/badge.svg?token=5LDR4B1O7Z
Expand Down
2 changes: 1 addition & 1 deletion codecov.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ comment: # this is a top-level key
require_changes: false # if true: only post the comment if coverage changes
require_base: true # [true :: must have a base report to post]
require_head: true # [true :: must have a head report to post]
after_n_builds: 10
after_n_builds: 14
coverage:
status:
patch:
Expand Down

0 comments on commit 2ebdf6a

Please sign in to comment.