typst / hayagriva

Rusty bibliography management.
Apache License 2.0
289 stars 44 forks source link

Fix #142 #145

Closed DerDrodt closed 2 months ago

DerDrodt commented 3 months ago

Fix #142.

Page ranges are now RangeInclusive<i32>, which fits the semantics of a page range better. Also changes how single pages are turned to ranges: 4 now becomes the inclusive range 4..=4, which fits.

Also, in citationberg, ranges where start and end are the same only render start.

Depends on typst/citationberg#12.

This is just a quick fix until I find time to rework page ranges.

reknih commented 3 months ago

typst/citationberg#142 is merged, so you can ~now add a patch to the Cargo.toml so we can merge this with passing CI.~

Edit: Change the dep to a Git dependency instead so it can be used in Typst, patches are ignored by upstream dependents.

reknih commented 2 months ago

Thank you!