-
Notifications
You must be signed in to change notification settings - Fork 2
Description
Following up on @chrisburr's nice PR #3 I noticed that some of the feedstocks where the name of the feedstock doesn't match the package the feedstock name is rendered incorrectly.
Example:
Lines 13 to 14 in e999c44
| [](https://github.com/conda-forge/ff-feedstock) | [](https://github.com/conda-forge/ff-feedstock) | [](https://anaconda.org/conda-forge/ff-static) | [](https://anaconda.org/conda-forge/ff-static) | [](https://anaconda.org/conda-forge/ff-static) | | | |
| [](https://github.com/conda-forge/ff-static-feedstock) | [](https://github.com/conda-forge/ff-static-feedstock) | [](https://anaconda.org/conda-forge/ff-static) | [](https://anaconda.org/conda-forge/ff-static) | [](https://anaconda.org/conda-forge/ff-static) | | |
which gives
Feedstock | Output | Downloads | Version | Platforms | Open PRs |
---|---|---|---|---|---|
should just be pointing to https://github.com/conda-forge/ff-feedstock as there is only one output ff-static
Feedstock | Output | Downloads | Version | Platforms | Open PRs |
---|---|---|---|---|---|
So
Line 5 in e999c44
"ff-static", |
should be removed as ff-static
was archived given a naming mistake by me (one could imagine a future in which FF has a shared library ff
output in addition to the existing ff-static
output).
Similarly for
Line 8 in e999c44
"looptools", |
should just be pointing to https://github.com/conda-forge/looptools-feedstock with the output of looptools-static
.
There might be a few others.
Also
.github/.github/workflows/update-readme.yml
Lines 4 to 6 in e999c44
# Run every 6 hours | |
schedule: | |
- cron: "0 */6 * * *" |
feels like it might get a bit noisy, so maybe we should lower that to daily? I guess this would really only get updates if there a new PRs opened up, given that it only commits changes
if [ -n "$(git status --porcelain)" ]; then |
but that could easily happen multiple times a day some days just given release update PRs.
Thoughts from others?