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

Fix bug in citar-citeproc-select-csl-style #815

Merged

Conversation

benthamite
Copy link
Contributor

Since list is a list of cons cells, to get the name of the csl style when list has one element caar needs to be used; car returns the cons cell itself.

@benthamite benthamite changed the title Fix bug in citar-citeproc-select-csl-style Fix bug in citar-citeproc-select-csl-style Dec 26, 2023
@benthamite
Copy link
Contributor Author

Separately (as I just noticed), I think the prompt on line 87 should be "Select CSL style: ", since the user is prompted to select a style, not a file.

@bdarcus
Copy link
Contributor

bdarcus commented Dec 26, 2023

Separately (as I just noticed), I think the prompt on line 87 should be "Select CSL style: ", since the user is prompted to select a style, not a file.

Could you please make that change while you're at it?

@benthamite
Copy link
Contributor Author

Yep, done.

@bdarcus bdarcus merged commit 885b86f into emacs-citar:main Dec 26, 2023
6 checks passed
@bdarcus
Copy link
Contributor

bdarcus commented Dec 26, 2023

Thanks!

I did squash the commits so I could easily rewrite the commit message (though I realize there's a remaining mistake; the correction is to the prompt, rather than the docstring)

https://www.conventionalcommits.org

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.

2 participants