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

Implement printing initials in xnames #49

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

goerz
Copy link

@goerz goerz commented Mar 6, 2023

Implement xnames(entry; names=:last) to print first name initials (as stated in the documentation) and xnames(entry; names=:lastonly) to print last names only.

Add an option and to use put " and " between the last two names.

Closes #48

Implement `xnames(entry; names=:last)` to print first name initials (as
stated in the documentation) and `xnames(entry; names=:lastonly)` to
print last names only.

Add an option `and` to use put " and " between the last two names.

Closes Humans-of-Julia#48
@Azzaare Azzaare added this to In progress in Modern Academics via automation Mar 6, 2023
@Azzaare Azzaare added this to the v1.0 milestone Mar 6, 2023
@goerz
Copy link
Author

goerz commented Mar 9, 2023

I don't know what's going on with the failing test… I'm seeing the same failure locally on my macBook, but they're failing for the current master branch too, so I don't think it's a defect with the PR

@Azzaare
Copy link
Member

Azzaare commented Mar 31, 2023

Sorry for the long silence.

I think we test that the output is similar to the content of some files. Probably, I need to regenerate those files to fit your changes.

I will have a look next week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Modern Academics
In progress
Development

Successfully merging this pull request may close these issues.

xnames(entry; names=:last) does not include initials
2 participants