kprussing / pandoc-acro

Pandoc filter for managing acronyms
BSD 2-Clause "Simplified" License
9 stars 4 forks source link

Support short/long-plural-form #9

Closed UFOMelkor closed 1 year ago

UFOMelkor commented 1 year ago

As this is my first contact with python, you might have a closer look at the code ;-)

UFOMelkor commented 1 year ago

One thing I noticed while implementing: The markdown and the LaTeX renderer differ when rendering an initial plural: Markdown Betriebsräte (BR) vs Betriebsräte (BRs) LaTeX LaTeX uses the plural abbreviation after a plural form.

kprussing commented 1 year ago

Overall I'd say it looks good. You were right that the changes were small.

Per your second comment. If this is a widespread error, then go ahead and open a new issue for tracking purposes. (Whatever acro does is assumed "correct" as far as this filter is concerned)