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 typo: use_safetenstors to use_safetensors #31184

Merged
merged 1 commit into from
Jun 3, 2024

Conversation

CharlesCNorton
Copy link
Contributor

Corrected a typo in security.md. Changed use_safetenstors to use_safetensors in the section discussing the usage of safe formats for loading models to prevent arbitrary code execution.

What does this PR do?

This PR corrects a typo in the security.md file. The typo was found in the section discussing the usage of safe formats for loading models to prevent arbitrary code execution. The incorrect term use_safetenstors was corrected to use_safetensors.

Fixes # (issue)

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you read the contributor guideline, Pull Request section?
  • Was this discussed/approved via a Github issue or the forum? Please add a link to it if that's the case.
  • Did you make sure to update the documentation with your changes? Here are the documentation guidelines, and here are tips on formatting docstrings.
  • Did you write any new necessary tests?

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag members/contributors who may be interested in your PR.

Models:

Library:

Integrations:

Documentation: @stevhliu and @MKhalusova

HF projects:

Maintained examples (not research project or legacy):

Corrected a typo in security.md. Changed `use_safetenstors` to `use_safetensors` in the section discussing the usage of safe formats for loading models to prevent arbitrary code execution.
Copy link
Contributor

@younesbelkada younesbelkada left a comment

Choose a reason for hiding this comment

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

Thanks !

Copy link
Collaborator

@amyeroberts amyeroberts left a comment

Choose a reason for hiding this comment

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

Hi @CharlesCNorton, thanks for fixing!

As a general note, please make sure to properly read, review and fill-out the PR template when opening. For example, all of the tick-box sections are selected here, which is not representative of the PR. One should also avoid tagging so many people, all of the people @'ed in the PR description would have received a notification, which isn't scalable if everyone did this. Select only one or two of the most relevant people to tag. If you're not sure - just select who you think might be best and they'll be happy to direct it to the correct person :)

@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.

@younesbelkada younesbelkada merged commit 97e5a70 into huggingface:main Jun 3, 2024
8 checks passed
zucchini-nlp pushed a commit to zucchini-nlp/transformers that referenced this pull request Jun 11, 2024
Corrected a typo in security.md. Changed `use_safetenstors` to `use_safetensors` in the section discussing the usage of safe formats for loading models to prevent arbitrary code execution.
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.

4 participants