| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## main #195 +/- ##
=======================================
Coverage 99.20% 99.20%
=======================================
Files 15 15
Lines 2507 2507
=======================================
Hits 2487 2487
Misses 20 20
|
Sorry, something went wrong.
|
Thanks for catching that @stevenhua0320 . That should have been catched by scikit-package. Please can you check that the scikit-package template is correct and make an issue if not? Thanks. |
Sorry, something went wrong.
|
@sbillinge Now I checked with the scikit-package repo and found that in the new convention of scikit-package we named the file as matrix-and-codecov-on-merge-to-main.yml. But in the previous workflow we name the diffpy.structure with matrix-and-codecov.yml, and in the package update command, since the materials in the file is the same, so it didn't change anything. But this arises the problem of inconsistency of workflow and the README.rst. Therefore, I don't think we need any edits there, but need to consider whether we need to rename the workflow file here and change it back to the original link at the next release. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
@sbillinge ready to review. Note that because in the new skpkg workflow we changed the name of the yaml file that related to CI, so now with the previous link in the README.rst the CI badge could not show up. Now we change to the correct workflow and should show up.