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 #31

Closed
3 tasks
hillaryj opened this issue Oct 27, 2020 · 3 comments
Closed
3 tasks

Add test coverage for python scripts #31

hillaryj opened this issue Oct 27, 2020 · 3 comments

Comments

@hillaryj
Copy link
Contributor

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
Copy link
Member

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
Copy link
Member

jsf9k commented 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

Issue moved to cisagov/development-tools #1 via ZenHub

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

3 participants