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

[HF] Let server decide default repo visibility #10

Merged
merged 2 commits into from
Dec 2, 2024

Conversation

Wauplin
Copy link
Contributor

@Wauplin Wauplin commented Nov 28, 2024

Until now, all repos were public by default when created without passing the private argument. This meant that passing private=False or private=None was strictly the same. This is not the case anymore. Enterprise Hub offers organizations to set a default visibility setting for new repos. This is useful for organizations forbidding public repos for security matters. This PR updates the default value to follow this principle.

Related to huggingface/huggingface_hub#2679 and #9

@Wauplin Wauplin changed the title Let server decide default repo visibility [HF] Let server decide default repo visibility Nov 28, 2024
@ncoop57
Copy link
Contributor

ncoop57 commented Nov 29, 2024

Hey @Wauplin , thanks so much for the PR!! However, since this is an nbdev project, development is a bit different than from what you are probably use to. You need to make changes to the jupyter notebooks and then export them in order for it to work properly. You can read more about it in this awesome CONTRIBUTING.md file (I'll add one here for future people!) https://github.com/AnswerDotAI/fasthtml/blob/main/CONTRIBUTING.md.

Could you please update your PR to work with nbdev?

@Wauplin
Copy link
Contributor Author

Wauplin commented Dec 2, 2024

@ncoop57 Done!

@ncoop57
Copy link
Contributor

ncoop57 commented Dec 2, 2024

Awesome, thanks so much @Wauplin !!

@ncoop57 ncoop57 merged commit 1405c84 into AnswerDotAI:main Dec 2, 2024
1 check failed
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