search

conda-incubator / conda-project

Tool for encapsulating, running, and reproducing projects with Conda environments
https://conda-incubator.github.io/conda-project/
BSD 3-Clause "New" or "Revised" License
28 stars 11 forks source link

chore: Add new static badge linking to documentation #116

Closed mattkram closed 1 year ago

mattkram commented 1 year ago

Closes #115

codecov[bot] commented 1 year ago

Codecov Report

Merging #116 (7917f78) into main (db904e1) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #116   +/-   ##
=======================================
  Coverage   97.68%   97.68%           
=======================================
  Files           9        9           
  Lines         819      819           
=======================================
  Hits          800      800           
  Misses         19       19

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

mattkram commented 1 year ago

I'm noticing that the main CI is running, even though I've updated the paths-ignore filter. There may be a bug in GitHub Actions: https://github.com/actions/runner/issues/2324

I'll create an issue to discuss whether we should instead use a path include filter, which I think will work.

mattkram commented 1 year ago

Disregard the last comment. There is no issue. Ironically, setting the ignore rules for the main.yaml changes the main.yaml, so I had to include that in the ignore rule for testing. Will revert before we merge this PR.

AlbertDeFusco commented 1 year ago

Is this still in draft or ready to go?

mattkram commented 1 year ago

I need to revert two commits. The other thing I realized is technically CI checks wouldn't pass in the future if the tests don't run (if only README is changed or something). I will make an issue for that. This one should be fine. Will revert those now.