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

Add subsequent Param to Name::is_supressed #3

Merged
merged 1 commit into from
Dec 11, 2023

Conversation

DerDrodt
Copy link
Contributor

@DerDrodt DerDrodt commented Dec 8, 2023

Will be used in fixing bug typst/hayagriva#111

This PR adds a parameter to Name::is_supressed, which allows a different behavior when writing names for a subsequent citation, per the name options in the CSL spec.

After merging, I will update hayagriva accordingly.

Will be used to fix typst/typst#2874
@reknih reknih merged commit 21dc215 into typst:main Dec 11, 2023
2 checks passed
@reknih
Copy link
Member

reknih commented Dec 11, 2023

Thank you!

(I updated the description in the PR so merging does not close typst/hayagriva#111 just yet)

@DerDrodt
Copy link
Contributor Author

(I updated the description in the PR so merging does not close typst/hayagriva#111 just yet)

Thank you for catching that!

@DerDrodt
Copy link
Contributor Author

One more quick question: To use this in hayagriva, citationberg needs a new release. Should it be a major one? This is technically a breaking change.

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.

None yet

2 participants