Skip to content

Conversation

@RichardLitt
Copy link
Contributor

I got annoyed recently that you assign editors, but you have to add reviewers. In my head, assigning and adding are the same thing.

I want a way where you can also assign a reviewer, without having to type another command. This adds that possibility, but it doesn't enforce it for others by documenting or showing it as an option - which might be a good idea. Instead, it just will fall back on adding the user if someone mistakenly uses the word 'assign'.

I got annoyed recently that you assign editors, but you have to add reviewers. In my head, assigning and adding are the same thing. 

I want a way where you can also assign a reviewer, without having to type another command. This adds that possibility, but it doesn't enforce it for others by documenting or showing it as an option - which might be a good idea. Instead, it just will fall back on adding the user if someone mistakenly uses the word 'assign'.
@xuanxu
Copy link
Member

xuanxu commented Apr 25, 2025

@RichardLitt Thanks!
I'm closing this PR because the changes it includes won't work, as they are changing a different responder class, but I've updated the code to allow assigning a reviewer, here: a65daa6

@xuanxu xuanxu closed this Apr 25, 2025
Copy link
Member

@xuanxu xuanxu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code used by JOSS is in the joss branch, not in main

@RichardLitt
Copy link
Contributor Author

@xuanxu I wasn't sure where to add it - this was my best guess. Thanks for passing that work forward to where it needs to go.

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