Skip to content

Conversation

@KirmTwinty
Copy link

I am submitting

  • a new package
  • an update for a package

Description: This project holds a touying template to write presentations for the French Université Gustave Eiffel, according to the graphic charter.

I have read and followed the submission guidelines and, in particular, I

  • selected a name that isn't the most obvious or canonical name for what the package does
  • added a typst.toml file with all required keys
  • added a README.md with documentation for my package
  • have chosen a license and added a LICENSE file or linked one in my README.md
  • tested my package locally on my system and it worked
  • excluded PDFs or README images, if any, but not the LICENSE
  • ensured that my package is licensed such that users can use and distribute the contents of its template directory without restriction, after modifying them through normal use.

@typst-package-check typst-package-check bot changed the title New touying template for French Université Gustave Eiffel touying-uge:0.1.0 Dec 4, 2025
@typst-package-check typst-package-check bot added the new A new package submission. label Dec 4, 2025
@elegaanz elegaanz self-assigned this Dec 5, 2025
@elegaanz
Copy link
Member

elegaanz commented Dec 8, 2025

Hey, there are some errors in your package that were reported by our CI system. Can you please fix them? Once that's done I'll proceed to a manual review. Let me know if you have questions or need help to fix these issues.

@KirmTwinty
Copy link
Author

Thank you!

It seems that now the issue is due to a svg file, a bit large, but I couldn't do better! Any idea?

Copy link
Member

@elegaanz elegaanz left a comment

Choose a reason for hiding this comment

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

I have some small remarks, after that it should be ready to merge.

Regarding the large SVG file, it's fine, the warning is here to bring it to attention in case something is wrong, but if it can't be compressed any further the current size is acceptable.

Copy link
Member

Choose a reason for hiding this comment

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

As explained in our docs the thumbnail should be a single page, even if I appreciate the effort of showcasing various slides here. At a lower resolution it wouldn't look too good I think. You can keep this image and re-use it to illustrate your README if you want (or use the individual pages without merging them).

Copy link
Member

Choose a reason for hiding this comment

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

Because this file is not necessary for users of your package, can you delete it, please?

Copy link
Member

Choose a reason for hiding this comment

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

Can you check that you are allowed to distribute these logos please? And can you specify in your README under which license/terms they are made available, unless they are under the GPL too?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new A new package submission.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants