Prepare for 1.9.0 Release

metrumresearchgroup / bbr

R interface for model and project management

https://metrumresearchgroup.github.io/bbr/

Other

23 stars 2 forks source link

Prepare for 1.9.0 Release #654

Closed barrettk closed 8 months ago

barrettk commented 8 months ago

Changeset: https://github.com/metrumresearchgroup/bbr/compare/1.8.1...release/1.9.0

barrettk commented 8 months ago

Apparently drone fails here (MPN oldest):

══ testthat results  ═══════════════════════════════════════════════════════════
--
242 | [ OK: 1013 \| SKIPPED: 16 \| WARNINGS: 17 \| FAILED: 1 ]
243 | 1. Error: summary_log() works correctly with nested dirs [BBR-SMLG-002] (@test-summary-log.R#42)

Not sure what's going on yet

kyleam commented 8 months ago

Hmm, that's an odd one, given build was green at time those summary_log came in. I can take a look in a bit if it doesn't jump out to you in the meantime.

https://github-drone.metrumrg.com/metrumresearchgroup/bbr/3810

barrettk commented 8 months ago

Yeah I was just looking through your PR for exactly that. Definitely odd

kyleam commented 8 months ago

Huh, PR build passed for this PR, so I'd first see if it is there on a restart:

barrettk commented 8 months ago

Huh, PR build passed for this PR, so I'd first see if it is there on a restart

Interestingly, that was failing too until I restarted it. I hadn't restarted the push CI, but will give that a try too. Edit: well it works now lol

barrettk commented 8 months ago

Scorecard Details

kyleam commented 8 months ago

so we'll just assume those CI failures we're transient Drone failures.

Yes, fingers crossed that those don't start popping up more frequently. I'm very pleased that we haven't seen many (if any) rare-but-regular-enough-to-be-painful transient failures in our packages.