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

Double period in references #4752

Closed
mattgarrish opened this issue Jul 8, 2024 · 1 comment · Fixed by #4780
Closed

Double period in references #4752

mattgarrish opened this issue Jul 8, 2024 · 1 comment · Fixed by #4780
Labels

Comments

@mattgarrish
Copy link

Description of problem

What happened (e.g., it crashed)?:

Very minor nit, but I noticed that generated references don't account for the last editor already having a period at the end of their name.

For example, you get two periods because of "Jr." in this reference:

CSS Cascading and Inheritance Level 5. Elika Etemad; Miriam Suzanne; Tab Atkins Jr.. W3C. 13 January 2022. W3C Candidate Recommendation. URL: https://www.w3.org/TR/css-cascade-5/

Expected behavior (e.g., it shouldn't crash):

If the last editor's name ends in a period, omit a second period.

@mattgarrish mattgarrish added the bug label Jul 8, 2024
@marcoscaceres
Copy link
Contributor

Offending line is here:

output += ". ";

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants