Skip to content

Disable checking file name and erroring for shared gdrive files. #518

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

Merged
merged 1 commit into from
Mar 13, 2025

Conversation

asolove
Copy link
Contributor

@asolove asolove commented Mar 14, 2024

This is a minimal version of fixing #517.

Whether this is a good idea or not is outside of my ability to judge.

Even if this is a good idea at a high level, there are two other things we might want to do:

  • Make the filename itself optional, so it's just not needed.
  • Update the source code of the current file to reflect the new name. This would prevent confusion but also introduces a surprising information leak if the file is renamed without the new name being safe to share.

@jpolitz
Copy link
Member

jpolitz commented Mar 13, 2025

Yeah this is a good idea, thanks, sorry it got lost for a while.

@jpolitz jpolitz merged commit 63d689d into brownplt:horizon Mar 13, 2025
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