Skip to content

Conversation

@birnstiel
Copy link
Contributor

I noticed that the format Firstauthor:secondauthor:year still only searches for the firstauthor and year. I modified the regular expression to include the entire colon-separated part and then split this into coauthors that are added to the queue if they exist. I tested this in a couple of cases and it seems to work and not break something, but please test yourself – hope I didn't break something.

@yymao
Copy link
Owner

yymao commented Dec 7, 2024

Thanks for the PR. I'll take a closer look and run some tests when I get time.

@yymao yymao changed the base branch from main to u/yymao/v0.6 December 8, 2024 17:27
@yymao
Copy link
Owner

yymao commented Dec 8, 2024

Thank you @birnstiel for the suggestion. I think this is a good idea, but I'd also like to preserve the original behavior as well because some people may use firstauthor:shorttitle:year: format, and this update will mess up those searches. I'll implement an option to toggle between these two options and release it in v0.6.0.

I'll merge your PR into the yymao:u/yymao/v0.6 branch and work on it further.

@yymao yymao merged commit 4d04e29 into yymao:u/yymao/v0.6 Dec 8, 2024
5 checks passed
@yymao
Copy link
Owner

yymao commented Dec 8, 2024

This PR has been incorporated in v0.6.0. Let me know if you encounter any issues @birnstiel !

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