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

add github issue option #10

Open
mpadge opened this issue Jul 8, 2020 · 7 comments
Open

add github issue option #10

mpadge opened this issue Jul 8, 2020 · 7 comments

Comments

@mpadge
Copy link
Member

mpadge commented Jul 8, 2020

Add option to add_contributors() function to open a github issue with dummy text to ping all contributors that they've been acknowledged in the README. Once that issue exists, triggering the option can then add any new contributors via additional automated issue comments.

mpadge added a commit that referenced this issue Aug 17, 2020
@mpadge
Copy link
Member Author

mpadge commented Aug 17, 2020

The above commit adds the initial functionality to open a new issue using the github client.

TODO

  • Add function to add more recent contributors to an existing issue
  • Add function to check whether gh is locally installed, and error if not
  • Handle issue opening when > 50 people are pinged, because github prevents more than that number being pinged from a single issue comment, so that case needs multiple comments.

mpadge added a commit that referenced this issue Aug 17, 2020
@mpadge mpadge closed this as completed in cf0fd2b Aug 18, 2020
@mpadge
Copy link
Member Author

mpadge commented Aug 18, 2020

Re-opening coz still have to do the 2nd of those , and also need to actually check that functionality works properly - it all appears to work well with issue opening.

@mpadge mpadge reopened this Aug 18, 2020
@maelle
Copy link
Member

maelle commented Nov 5, 2020

Also, imagine I open an issue with a question and you end up telling me to rather open a discussion thread in a forum to get help. My contribution is then not really one. Could you label the issue as out-of-scope for the tool?

And regarding the original intent of this very issue, maybe it could be stated in the issue template instead?

@mpadge
Copy link
Member Author

mpadge commented Nov 5, 2020

Yes, it would be very easy to label the issue as "out-of-scope" or "non-contrib" or something, but then again if issues really are not considered relevant contributions then they can simply be deleted. That was my initial thought at least, and the reason why i opted to retain a simpler acknowledge-all-type approach. A question is still a form of interest in, and presumably use of, the software. I guess for me the real issue in thinking about that was how such a label could be applied in a way that wasn't overtly negative. Label filtering in this context must necessarily be a filtering out, which is an inherently negative activity, so labels would work best if they contained a negation, but that is then potentially insulting. I tried but couldn't think of suitable non-negative labels that retained general applicability ("enquiry" comes close, but even then issues may be opened simply to comment without enquiry).

Any advice would be greatly appreciated! The whole issue extraction is graphql, so adding a labels field to the query would be totally easy, and this would be no trouble at all to implement. I just would like to do it in the most appropriate way possible.


Finally, I'm not sure exactly what you mean by this:

And regarding the original intent of this very issue, maybe it could be stated in the issue template instead?

Could you please explain a bit more? Thanks

@maelle
Copy link
Member

maelle commented Nov 5, 2020

how would you delete an issue?

I was thinking the issue template could state there's a README section with contributions but then who reads that...

@mpadge
Copy link
Member Author

mpadge commented Nov 5, 2020

Magic new-ish "delete issue" button:

image

I was thinking the issue template could state there's a README section with contributions ...

I think that is what it already does:
https://github.com/mpadge/allcontributors/blob/126be32f5b51136099c2b176fd73253b139c4bb4/R/add-contributors.R#L377-L382

That's what you mean, right?

@maelle
Copy link
Member

maelle commented Nov 5, 2020

aaah cool that one can delete issues.

by issue template I mean .github/ templates, that one sees when opening an issue.

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

No branches or pull requests

2 participants