Skip to content
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

ieee-monolith:0.1.0 #1537

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

ieee-monolith:0.1.0 #1537

wants to merge 3 commits into from

Conversation

Fricsion
Copy link

@Fricsion Fricsion commented Jan 9, 2025

I am submitting

  • a new package
  • an update for a package

Description: Explain what the package does and why it's useful.
I like IEEE style of references/bibliography.

And I wanted it to be single column instead of two.

Credit: Typst GmbH (I used their template as the base for mine)

  • You can just copy-and-paste main.typ from your previously-written ieee-charged (by Typst Team) document. And it works!
    • Additional features are all optional. e.g. global font setting
  • Adjust spacings according to the single column setup.

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.

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.

Thanks for changing the name. Here a few more points I would like to discuss before merging.

packages/preview/ieee-monolith/0.1.0/template/main.typ Outdated Show resolved Hide resolved
@@ -0,0 +1,21 @@
MIT License

Copyright (c) 2024 Taiki M.
Copy link
Member

Choose a reason for hiding this comment

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

I'm no license expert, but because you based your work on charged-ieee, I feel like the original authors should be credited here too.

Copy link
Author

Choose a reason for hiding this comment

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

As far as I'm aware, charged-ieee uses MIT-0 license, which has no attribution. I'm no license expert either. But I think it should be fine.

Copy link
Author

@Fricsion Fricsion Jan 9, 2025

Choose a reason for hiding this comment

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

I chose MIT license solely because other template submitters have done so. But I'm open to changing the license from MIT to MIT-0 if your team prefer that way. It might cause less troubles.

packages/preview/ieee-monolith/0.1.0/typst.toml Outdated Show resolved Hide resolved
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.

2 participants