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

fix: usage of node12 which is deprecated in CI #1659

Conversation

hamirmahal
Copy link
Contributor

fixes #1658.

@@ -14,18 +14,13 @@ jobs:
uses: actions/checkout@v3

- name: Install Python
uses: actions/setup-python@v1
uses: actions/setup-python@v5
Copy link
Contributor Author

Choose a reason for hiding this comment

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

v5 is the latest major version of actions/setup-python at the time of this writing.

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Copy link
Collaborator

@ArthurZucker ArthurZucker left a comment

Choose a reason for hiding this comment

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

Thanks for updating!

Comment on lines -24 to -27
- name: Install Rust
uses: actions-rs/toolchain@v1
with:
toolchain: stable
Copy link
Collaborator

Choose a reason for hiding this comment

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

is there a reason why this one is deleted as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, that action also uses node12 which is deprecated.

@hamirmahal
Copy link
Contributor Author

You're welcome!

@ArthurZucker
Copy link
Collaborator

the install dependencies for node build seems to break from this 😓

@hamirmahal hamirmahal force-pushed the fix/usage-of-node12-which-is-deprecated-in-docs-check-actions branch from 84e8031 to 3e7f99b Compare October 25, 2024 17:56
@hamirmahal
Copy link
Contributor Author

It looks like hamirmahal@84e8031 fixes it.

At the very least, that change passes with all checks in a fork.

@ArthurZucker
Copy link
Collaborator

PR just merged to main fixed this! (it did all of the actions) 🤗 thanks for the contribution

@hamirmahal
Copy link
Contributor Author

You're welcome! Feel free to close this if it's no longer necessary.

@ArthurZucker
Copy link
Collaborator

Closing then! 🤗

@hamirmahal hamirmahal deleted the fix/usage-of-node12-which-is-deprecated-in-docs-check-actions branch November 4, 2024 17:44
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.

docs-check.yml uses node12 which is deprecated
3 participants