Skip to content

ctu-report:0.2.0 #2052

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

ctu-report:0.2.0 #2052

wants to merge 2 commits into from

Conversation

tomkys144
Copy link

I am submitting

  • a new package
  • an update for a package

Description: Adding template for reports based on CTU graphics manual.

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 added the new A new package submission. label Mar 20, 2025
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.

Regarding the various logos, do you have the rights to distribute them? And if so, is there any copyright or license information that you need to include?

Copy link
Member

Choose a reason for hiding this comment

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

Font files should not be copied to this repository. Can you please delete them?

@@ -0,0 +1,16 @@
[package]
name = "ctu-report"
Copy link
Member

Choose a reason for hiding this comment

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

If this is an official package, a member of the CTU staff should send an email to [email protected] saying they approve this package (using their institutional email address, of course). If it is unofficial, I would like you to change the name for something more distinctive: if someone else makes an alternative to this package, they shouldn't be disadvantaged because you reserved the canonical name. Generally, people add an adjective to make their package name more unique, but you can get creative and make a pun or some indirect reference to the CTU if you want.

@elegaanz
Copy link
Member

elegaanz commented May 5, 2025

Hello, just a message to remind you of the review I left a few weeks ago. If you don't have time to handle it for now, please let me know, I'll close this PR for inactivity (but then you can still submit your package again in another PR once it is ready).

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