Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add test coverage for python scripts #40

Open
3 tasks
hillaryj opened this issue Dec 1, 2020 · 1 comment
Open
3 tasks

Add test coverage for python scripts #40

hillaryj opened this issue Dec 1, 2020 · 1 comment

Comments

@hillaryj
Copy link
Contributor

hillaryj commented Dec 1, 2020

@hillaryj commented on Tue Oct 27 2020

This repository has no actual code testing, a .coveragerc file, or a test workflow, despite containing python scripts in the project_setup/scripts directory and requiring coverage in setup.py.

Acceptance Criteria

  • Add test coverage for scripts
  • Add .coveragerc file
  • Integrate with coveralls

@mcdonnnj commented on Wed Oct 28 2020

I think we should discuss separating the guide from the tools. It would be nice if the tools were their own package to install instead in my opinion. This would make it easier to add new tools and have a functional CI/CD setup for those tools.


@jsf9k commented on Wed Oct 28 2020

I think we should discuss separating the guide from the tools. It would be nice if the tools were their own package to install instead in my opinion. This would make it easier to add new tools and have a functional CI/CD setup for those tools.

I think this is a grand idea. Do you want to create an issue so we don't lose track of this?

@hillaryj
Copy link
Contributor Author

hillaryj commented Dec 1, 2020

This is blocked pending the move of the tools and scripts mentioned in #33

@hillaryj hillaryj transferred this issue from another repository Dec 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant