Skip to content

Conversation

3w36zj6
Copy link
Member

@3w36zj6 3w36zj6 commented Sep 20, 2025

@3w36zj6 3w36zj6 requested a review from Copilot September 20, 2025 03:43
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR makes the SSG favicon replaceable by allowing users to provide their own favicon.png file instead of using a hardcoded one.

  • Adds documentation explaining that users can provide a custom favicon.png in the public directory
  • Updates gitignore to exclude the favicon.png file from version control
  • Modifies the symlink task to create a symbolic link for the favicon.png file

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.

File Description
website/typst-docs-web/README.md Documents the new favicon customization feature
website/typst-docs-web/.gitignore Excludes favicon.png from version control
mise.toml Adds favicon.png symlink creation to the build task

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@3w36zj6
Copy link
Member Author

3w36zj6 commented Sep 20, 2025

Could you please do a quick review? @YDX-2147483647

Copy link
Contributor

@YDX-2147483647 YDX-2147483647 left a comment

Choose a reason for hiding this comment

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

People who run typst-docs-web for the first time may not prepare a favicon, but it can still be built without a favicon (just a few 404 errors). So I think it's no problem.

@3w36zj6 3w36zj6 merged commit 76c9646 into main Sep 21, 2025
5 checks passed
@3w36zj6 3w36zj6 deleted the feature/make-favicon-replaceable branch September 21, 2025 04:26
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.

5 participants