tower icon indicating copy to clipboard operation
tower copied to clipboard

chore: update GitHub Actions CI

Open striezel opened this issue 2 years ago • 2 comments

The following updates are performed:

Still using the outdated / unmaintained actions will generate several warnings in CI runs, for example in https://github.com/tower-rs/tower/actions/runs/4995991400:

Node.js 12 actions are deprecated. Please update the following actions to use Node.js 16: actions-rs/toolchain@v1, actions-rs/cargo@v1. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/.

The PR will get rid of those warnings.

striezel avatar Jun 10 '23 08:06 striezel

wasn't aware of this change in the ecosystem, but double checking docs at https://github.com/dtolnay/rust-toolchain and other info I found seems to back the incentive behind this PR.

In the mean time, the whole actions-rs organization has been archived. If the lack of maintainer activity in the repos over there was not enough of a hint that nothing is going to happen there anymore, then that step finally settled it for good.

So yeah ... I think everyone who is still using actions-rs/toolchain etc. should move to an alternative as soon as possible.

striezel avatar Jan 29 '24 13:01 striezel

@hawkw @GlenDC Any updates on this one? Looks like some of the checks have not even started yet, so I guess that maybe someone has to approve them in order for them to run?

striezel avatar Feb 26 '24 23:02 striezel

@tobz this is also a trivial one to merge, no? Or am I missing something?

GlenDC avatar Jul 22 '24 11:07 GlenDC

Yeah, this is probably fine to merge as-is... although it's another case where the PR branch is out-of-date and I can't trivially update it.

tobz avatar Jul 22 '24 13:07 tobz

although it's another case where the PR branch is out-of-date and I can't trivially update it.

It's been over a year, so that's no surprise. I almost got the impression the project had been abandoned due to the lack of activity here.

Anyway, I rebased the branch against the current master so this should be easier to merge now.

striezel avatar Jul 23 '24 03:07 striezel