-
Notifications
You must be signed in to change notification settings - Fork 83
Modified copyright, privacy, terms in docs #1160
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -150,7 +150,7 @@ def __call__(self, filename): | |
| master_doc = "index" | ||
|
|
||
| # General information about the project. | ||
| copyright = "2023-present, TorchCodec Contributors" | ||
| copyright = "©2023 Meta Platforms, Inc" | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Meta Open Source policy has a script regex that requires the copyright symbol to appear after "Copyright." This will result in a double copyright symbol effectively and render as Alternatively, we could replace the footer's copyright section using JavaScript |
||
| author = "Torch Contributors" | ||
|
|
||
| # The language for content autogenerated by Sphinx. Refer to documentation | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Currently, we use pytorch-sphinx-theme, which have these links to privacy and terms in raw html. We can't directly override it in our layout.html. As such, we would have to use javascript to replace those links.
We could also ask pytorch-sphinx-theme if they could wrap these links in an overridable block but Im not sure if it makes sense for them to support that.