-
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
Update to use "time" not "step" in dust 0.12.x #222
Conversation
Codecov ReportBase: 100.00% // Head: 100.00% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## master #222 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 29 29
Lines 2851 2834 -17
=========================================
- Hits 2851 2834 -17
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
This was read for review, until the that last version bump, which looks to have done something terrible to the version of dust installed; looks like remotes decided it would "upgrade" dust to a previous version on the logs. I'll open a separate PR that updates this to new actions, but in the meantime I think this is still ok to look at |
Fairly robotic, but also fairly painful. Once this is passing there are a couple of rough edges that can be smoothed over as this package was already having a bit of a fight with both step and time being used