python-project-template

python-project-template copied to clipboard

Keep workflows active

4

Github will disable workflows for inactive projects, and a project goes inactive after 60 days with no commits. This will stop smoke tests from detecting breaking changes from upstream package...

delucchi-cmu

Currently the LF-PPT version check pre-commit hook will *always* pass, even if the template that a project uses is out of date. Given that many people don't use pre-commit locally,...

drewoldag

It would be great to add `check-yaml` hook from the "standard" pre-commit repo. It runs fast and very useful for users who are not so familiar with this format, but...

hombit

We currently have a breadcrumb README in the `.../docs/notebooks` directory when a new project is hydrated. I'm suggesting that we add more and perhaps use those READMEs as a place...

drewoldag

Potential duplicate stage in asv workflow

2

@OliviaLynn pointed out in review of https://github.com/astronomy-commons/hipscat/pull/124 that the asv-pr and asv-main scripts have two stages with action like: `Cache Python ${{ env.PYTHON_VERSION }}` Is this duplication intentional and necessary?

delucchi-cmu

Add code coverage calculation to pre-commit stages

2

I'm bummed when everything in pre-commit passes locally, then the github workflow lets me know that there's one new line that doesn't have test coverage. Always aiming for 100% is...

delucchi-cmu

Confirm documentation for "incorporating into prior project"

1

This is feedback received from a user updating a pre-existing project with the PPT: > The directory layout described in "Incorporating the template into a prior project" is different than...

drewoldag

Fail the ASV workflows if at least one of benchmarks deteriorates above a certain threshold (e.g. 150% / 200%). **asv-main.yml**: - Execute `asv run` without `|| true`, allowing the command...

camposandro

Codecov in private repositories

1

Codecov action doesn't work for private repositories unless Codecov token is added. However action doesn't fail because "fail_ci_if_error: true" is not specified and upload failure is implicit. I suggest: -...

hombit

This is feedback received from a user updating a pre-existing project with the PPT: > The LINCC Frameworks template does not impose a rigid directory structure. However, some of the...

drewoldag