Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update translate workflow #4027

Closed
wants to merge 3 commits into from
Closed

Update translate workflow #4027

wants to merge 3 commits into from

Conversation

hankolsen
Copy link
Contributor

@hankolsen hankolsen commented Oct 11, 2022

This PR updates actions used in the translations.yml workflow.

Github will start running all actions on Node 16, so actions using Node 12 need to be update to use Node 16.

This PR updates:

  • actions/checkout v2 -> v3
  • volta-cli/action v1 -> v4

Remove the action actions-rs/toolchain as it has been abandoned. Install Rust "manually" instead.


This change is Reviewable

@hankolsen hankolsen force-pushed the update-translate-workflow branch from 1067909 to 20b7543 Compare October 18, 2022 10:59
@hankolsen hankolsen force-pushed the update-translate-workflow branch from 20b7543 to eedc4eb Compare November 7, 2022 11:25
@hankolsen hankolsen force-pushed the update-translate-workflow branch from eedc4eb to 93ca305 Compare November 7, 2022 11:28
@hankolsen hankolsen marked this pull request as ready for review November 7, 2022 11:32
@hankolsen hankolsen requested a review from raksooo November 7, 2022 11:34
Copy link
Contributor Author

@hankolsen hankolsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion


.github/workflows/translations.yml line 39 at r1 (raw file):

            - name: Install nightly Rust
              run: curl https://sh.rustup.rs -sSf | sh -s -- -y

Antoher alternative could be to use https://github.com/dtolnay/rust-toolchain

Copy link
Contributor Author

@hankolsen hankolsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion


.github/workflows/translations.yml line 39 at r1 (raw file):

Previously, hankolsen wrote…

Antoher alternative could be to use https://github.com/dtolnay/rust-toolchain

Original idea came from: actions-rs/toolchain#219 (comment)

@hankolsen hankolsen marked this pull request as draft November 7, 2022 13:05
@raksooo raksooo closed this Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants