search

18F / pulse

How the federal .gov domain space is doing at best practices and policies.
Other
94 stars 56 forks source link

CircleCI config #778

Closed buckley-w-david closed 6 years ago

buckley-w-david commented 6 years ago

PR to add CircleCI configuration (as well as make some changes to the dependencies)

As with #777 this is a pretty "CDS specific" addition, if you don't feel it belongs in 18Fs repo, feel free to ❌

konklone commented 6 years ago

Could you describe what the CI additions will do?

buckley-w-david commented 6 years ago

Unless 18F sets up CircleCI with their repo, this change will do nothing.

If however you were to do that, it would run all the tests in the tests folder using pytest for every push, and report the success/failure on github next to the commit, and in any PRs.

konklone commented 6 years ago

We use CircleCI across 18F/TTS, and that's some pretty reasonable stuff, so I'm in. :+1: