-
Notifications
You must be signed in to change notification settings - Fork 3
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
4 scripts to ingest all required data #5
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #5 +/- ##
======================================
Coverage ? 56.09%
======================================
Files ? 2
Lines ? 123
Branches ? 0
======================================
Hits ? 69
Misses ? 54
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Heads up @daniel-coles-ons - you see this message the first time you configure codecov. It attempts to give you a coverage update report (eg x more lines covered since this branch diverged from main). It has nothing to compare to the first time it runs, but from thereon you will get coverage deltas appearing in the comments from codecov bot. |
Closes #4, #3
Description
Fixes #(issue_number)
Motivation and Context
Type of change
How Has This Been Tested?
One test unit for StaticDataIngest. Other tests to follow once CodeCov trigger rectified.
Test configuration details:
Advice for reviewer
Ingest scripts for all NAPTAN, geography and timetable data
Ingest scripts for 1 minute sample of BODS realtime data
Updated README
Various bug fixes/updates
Single unit test
CodeCov pending...
Checklist:
Additional comments