Skip to content

Use bundled fonts in the documentation #26247

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
Jul 30, 2025

Conversation

nadavtzaysler
Copy link
Contributor

(X ) This is not user-visible or is docs only, and no release notes are required.

@cla-bot cla-bot bot added the cla-signed label Jul 20, 2025
@nadavtzaysler nadavtzaysler requested a review from mosabua July 20, 2025 20:48
@github-actions github-actions bot added the docs label Jul 20, 2025
@nadavtzaysler nadavtzaysler requested a review from wendigo July 20, 2025 20:48
@nadavtzaysler
Copy link
Contributor Author

will solve the same issue in #26031 but for the trino docs

@chenjian2664
Copy link
Contributor

@wendigo wendigo force-pushed the add-local-fonts-to-docs branch from 4aec5a7 to 1c71ce4 Compare July 28, 2025 08:28
@wendigo wendigo changed the title added local fonts into the trino docs to stop docs not loading when not connected to internet Use bundled fonts in the documentation Jul 28, 2025
@wendigo wendigo force-pushed the add-local-fonts-to-docs branch from 1c71ce4 to 84634c1 Compare July 28, 2025 08:30
@nadavtzaysler
Copy link
Contributor Author

nadavtzaysler commented Jul 28, 2025

Seems like the docs are also trying to access github.com which can also be hurtful in a private domain, I will implement a way to check it before trying to acces it, @wendigo would you like it to be in a new pull request or should i add the changes here?

@wendigo
Copy link
Contributor

wendigo commented Jul 28, 2025

@nadavtzaysler no. There is no point in doing that

@nadavtzaysler nadavtzaysler requested a review from wendigo July 29, 2025 19:39
@wendigo wendigo force-pushed the add-local-fonts-to-docs branch from d00335f to fcbcc15 Compare July 30, 2025 06:39
@wendigo
Copy link
Contributor

wendigo commented Jul 30, 2025

I've dropped last commit. Please create separate change for it.

@wendigo wendigo merged commit c9e26f6 into trinodb:master Jul 30, 2025
8 checks passed
@github-actions github-actions bot added this to the 477 milestone Jul 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants