Skip to content

invicta-thesis:1.0.0 #2738

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 3 commits into from
Jul 23, 2025
Merged

invicta-thesis:1.0.0 #2738

merged 3 commits into from
Jul 23, 2025

Conversation

Tonevanda
Copy link
Contributor

I am submitting

  • a new package
  • an update for a package

Description: Master's Thesis template for FEUP students.

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.

@elegaanz elegaanz closed this Jul 22, 2025
@elegaanz elegaanz reopened this Jul 22, 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.

Thank you for submitting this package. Can you move the preview folder you created inside of packages, merging it with the existing packages/preview one, please?

@@ -0,0 +1,16 @@
[package]
name = "feup-thesis"
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 template is not official, I will ask you to change the name to something less canonical, to respect our naming guidelines.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done 👍

Copy link
Member

Choose a reason for hiding this comment

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

If there is a particular copyright or license for this file (and the other logo), can you please state so in your README — or some other file that the user can easily find.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thank you for pointing that out. i added it to the note at the end of the README

@elegaanz elegaanz self-assigned this Jul 22, 2025
@typst-package-check typst-package-check bot changed the title feup-thesis:1.0.0 invicta-thesis:1.0.0 Jul 22, 2025
@typst-package-check typst-package-check bot added the new A new package submission. label Jul 22, 2025
@Tonevanda Tonevanda requested a review from elegaanz July 22, 2025 21:57
@Tonevanda
Copy link
Contributor Author

should be good to go i think. thank you for the feedback @elegaanz

@elegaanz
Copy link
Member

Perfect, thank you!

@elegaanz elegaanz merged commit e4d8e48 into typst:main Jul 23, 2025
3 checks passed
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