-
Notifications
You must be signed in to change notification settings - Fork 9
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 make targets for cleaning db and inserting fake data. #122
Conversation
Codecov Report
@@ Coverage Diff @@
## master #122 +/- ##
==========================================
- Coverage 94.57% 94.49% -0.09%
==========================================
Files 107 107
Lines 4570 4576 +6
Branches 258 260 +2
==========================================
+ Hits 4322 4324 +2
- Misses 208 211 +3
- Partials 40 41 +1
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This reverts commit 56690b9.
That is both terrible practice and wont work because you have to pass coverage on the latest commit. |
this is common practice |
This reverts commit 0cc9936.
This reverts commit 71d8c08.
* Add make targets for cleaning db and inserting fake data. * Update Makefile * Set Plugin logs to INFO * Add 'docopt' to dev-dependencies * Update 'fake' script * Update __init__.py * Remove clean_db.py * Update black line length (what's scope?) * Remove clean_db.py * Format scripts/fake * Update scripts.fake * Split long SQL statement * Update __main__.py * Update __main__.py * Update __main__.py * fix clean db Co-authored-by: Jeremiah Boby <mail@jeremiahboby.me>
Adds make targets insert-data, insert-bulk-data, clean-db, and clean-test. Also speeds up the data insertion by roughly 2x