Conversation

Fdawgs

Bumps Actions to their latest major versions. Main change is they're now using Node 16 instead of 12, which was EOL at end of April.

@brianc

Thanks for doing this! Definitely want to upgrade but....hmmmm any idea why all the CI checks failed?

@charmander

@brianc One of stream-tester’s transitive dependencies. It’s been failing the same way on current master for a while.

@Fdawgs

One of stream-tester’s transitive dependencies. It’s been failing the same way on current master for a while.

Thanks @charmander, had looked at the errors thrown and couldn't tell why it was failing.

@brianc

One of stream-tester’s transitive dependencies. It’s been failing the same way on current master for a while.

barf. going to just kill this dep & remove stream tester. hasn't really provided much value anyway. I'll get that ed up, and then unfortunatley ask PR contributors to rebase on master once I remove this broken dep. Wish made that busy-work easier to do behind a button like "bring PR up to date with merge target" or something.

@briancbrianc mentioned this pull request May 10, 2022
@brianc

@Fdawgs thanks for doing this! I went ahead and bumped it on #2743 because I was trying to fix CI from being broken so this isn't needed anymore, but thanks for contributing...your bumping this forward helped me start pulling on the thread!

@briancbrianc closed this May 11, 2022
@Fdawgs

Great, thanks @brianc!

@FdawgsFdawgs deleted the -1 branch May 11, 2022 18:31
Sign up for free to join this conversation on . Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.