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

kinase 0.1.0 #590

Merged
merged 6 commits into from May 16, 2024
Merged

kinase 0.1.0 #590

merged 6 commits into from May 16, 2024

Conversation

indicatelovelace
Copy link
Contributor

@indicatelovelace indicatelovelace commented Apr 24, 2024

I am submitting

  • a new package
  • an update for a package

Description: This packages allows the easy styling of document links. While Typst produces correct output for internal and external links, there is no easy way to style those. To achieve this the user can set styling rules for different keys. Keys are regex or strings. The package includes regex helpers for mails, phone numbers and urls.

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.

@laurmaedje laurmaedje added the new A new package submission. label Apr 29, 2024
Copy link
Member

@laurmaedje laurmaedje left a comment

Choose a reason for hiding this comment

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

The package is called "kinase", but the PR says "Kinave". which one is correct?

authors = ["Lennart Schuster"]
license = "MIT"
description = "Easy styling for different link types like mails and urls."
exclude = ["docs/manual.pdf", "ressources/example.png"]
Copy link
Member

Choose a reason for hiding this comment

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

I think docs & resources could be excluded completely. ressources should likely also be resources (typo).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think docs & resources could be excluded completely. ressources should likely also be resources (typo).

Changed it.

Copy link
Contributor

Choose a reason for hiding this comment

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

It still says ressources with two s, is it intentional?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nah english is the only language to spell it that way, I fixed it.

@indicatelovelace indicatelovelace changed the title Kinave 0.1.0 kinase 0.1.0 May 3, 2024
@indicatelovelace
Copy link
Contributor Author

The package is called "kinase", but the PR says "Kinave". which one is correct?

Typo on my end. The package is named after the enzyme kinase. I also changed the title of the pull request to match the case.



```typ
#import "@preview/link-style:0.0.1"
Copy link
Contributor

Choose a reason for hiding this comment

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

I think that import uses an old name and version and should be updated.

@@ -0,0 +1,27 @@
# Kinave
Copy link
Contributor

Choose a reason for hiding this comment

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

There should only be a sub-folder per version in this directory, so I think you should delete this file.

@@ -0,0 +1,16 @@
#import "../bib.typ": *
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the renaming of the directory didn't go as expected, the old one still exists here.

@@ -0,0 +1,16 @@
#import "../bib.typ": *
Copy link
Contributor

Choose a reason for hiding this comment

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

Ideally, this import should not be relative but use the package specification. If you want to keep it relative, it should probably be ../lib.typ.

Suggested change
#import "../bib.typ": *
#import "@preview/kinase:0.1.0": *

@indicatelovelace
Copy link
Contributor Author

The issues should be fixed, thanks for being so thorough.

@reknih reknih force-pushed the main branch 6 times, most recently from 92aab43 to d06d87c Compare May 15, 2024 10:12
@elegaanz elegaanz merged commit 47dc82a into typst:main May 16, 2024
1 check passed
@elegaanz
Copy link
Contributor

Thank you :)

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.

None yet

3 participants