try to deal with arrow 0.15.x's changed `.get()`

mozilla-releng / beetmoverscript

An implementation of scriptworker for moving TC artifacts to all release-related buckets.

1 stars 19 forks source link

try to deal with arrow 0.15.x's changed `.get()` #255

Closed escapewindow closed 5 years ago

escapewindow commented 5 years ago

I'd like to land https://github.com/mozilla-releng/build-puppet/pull/599 but don't want to break anything with the new arrow.

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 1014

7 of 7 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 98.752%

Totals
Change from base Build 1011:	0.01%
Covered Lines:	791
Relevant Lines:	801

💛 - Coveralls

escapewindow commented 5 years ago

rail 15:05 @escapewindow why do we need to split by "/" in that arrow patch?

aki 15:06 @rail arrow didn't like that YYYY and MM were repeated in the string so i'm just taking the last portion guessing that it could lead to something like 2018/10/2019-09-... and then it has conflicting data about what to use rail 15:08 oh, I see, we tried to convert the path into a date :slightly_smiling_face: aki 15:08 yes