:wave: Hello, team! This is the first time I made ...
# dev-metaflow
r
👋 Hello, team! This is the first time I made contributions to open source. I fixed the bug https://github.com/Netflix/metaflow/issues/816 and created a pull request.
✅ 1
a
Thanks for the PR @red-caravan-38397! We will take a look shortly.
r
Thank you!
a
@red-caravan-38397 I have added a comment to your PR.
r
I will work on it again. Thank you for your feedback!
@square-wire-39606 Please forgive me. I was so excited that I forgot to change the parameter for
TaskDataStore
. Please review it again. What else do I need to do?
a
Great. Just added a minor comment. If you can address that, we can merge this PR in.
r
@square-wire-39606 Looks like the tests failed in the R test step with an error message saying
there is no package called 'usethis'
which doesn't make sense to me. Do I need to do anything about it? Thanks.
a
let me look. most likely, we can address that as a separate PR