Fix action errors when codecov is "n" and replaced hardcoded names with placeholders #21
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR Checklist
Not sure how to do that as it already throws an error on github actions when creating a repo without codecov but that kind of testing is not part of the current framework and would require something like act to test the actions.
Alternatively one could use a script that extracts the run command. But I am not sure if that is very useful in the end as it can't take all previous steps into account.
docs
is updatedDescription of changes
The Contributing.md contained a reference to Cookiecutter PyPackage which is replaced with an appropriate placeholder to avoid that adapting this section is overlooked by someone.
Also the hard-coded name of the maintainer in mkdocs.yml was replaced with a placeholder.
The "Run tests" action step in main.yml is changed such that it runs successfully also without codecov dependencies installed.
Docs are rewritten to make clear that no coverage report is created without codecov set to "y"