-
Notifications
You must be signed in to change notification settings - Fork 2
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
Descriptions + release dates #346
Descriptions + release dates #346
Conversation
eb11b1e
to
8ed4cd6
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #346 +/- ##
==========================================
- Coverage 82.50% 82.24% -0.26%
==========================================
Files 87 87
Lines 2669 2636 -33
Branches 275 275
==========================================
- Hits 2202 2168 -34
Misses 404 404
- Partials 63 64 +1 ☔ View full report in Codecov by Sentry. |
9c63c94
to
6b2de17
Compare
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.
Few small things to check with the dates of datasets but otherwise good.
Import all script super helpful.
There is a separate, more up-to-date fuel import dataset that exists in the app, and this import isn't used. The test for this import is also removed.
6b2de17
to
e1e4d72
Compare
Also need to change the templates to show release dates, and change source attributions |
e1e4d72
to
e021a98
Compare
This commit covers updates to all of the dataset import scripts, ensuring that their release date, description, and source label are what is expected (according to the 'crowdsourcing' sheet).
This change also removes part of the MP import script that runs all the MP-related scripts, to avoid redundancy
e021a98
to
b724241
Compare
Fixes #192 and fixes #304