Skip to content
This repository has been archived by the owner on Feb 28, 2024. It is now read-only.

Identity first-time contributors and external contributors #85

Open
pjcozzi opened this issue Oct 2, 2017 · 6 comments
Open

Identity first-time contributors and external contributors #85

pjcozzi opened this issue Oct 2, 2017 · 6 comments

Comments

@pjcozzi
Copy link

pjcozzi commented Oct 2, 2017

We like to tweet to congratulate and thank first-time contributors and external contributors. It would be useful for a comment to explicitly @mention the outreach team so they know to tweet.

Currently, it is possible to identify first-time contributors by filtering emails for "Welcome to the Cesium community", but I don't know if there is a unique way to filter for external contributors, but this would be easy for the concierge to check.

CC @slchow @vanessamoore

@pjcozzi
Copy link
Author

pjcozzi commented Nov 30, 2017

Bumping this since we like to tweet about all community contributions.

When a pull request from a community contributor is merged, the concierge could just @ mention someone(s) (configurable) and also say if it is a first time contributor.

Longer-term, if/when the concierge can label issues, it could label issues as "community contribution", "first-time contributor", etc. - which would be useful for stats as well.

@pjcozzi
Copy link
Author

pjcozzi commented Dec 28, 2018

@OmarShehata is this one on the short-term radar?

@OmarShehata
Copy link
Contributor

Yes, I ran into the same problem of checking the correct branch/fork as in this PR #118 so it'll be easy to do right after that one.

@OmarShehata
Copy link
Contributor

This is merged, and I have seen concierge reporting "Thanks for your first time contribution!" need to investigate why it didn't post for this one CesiumGS/cesium#7944.

@verybigzhouhai
Copy link

verybigzhouhai commented Jun 28, 2019

@OmarShehata Is it because the first PR I submitted before was shut down by myself, so it was not identified this time? CesiumGS/cesium#7911 ,When I shut down the pr, I received a message from the concierge.

@OmarShehata
Copy link
Contributor

I think it may be an error in my logic, that Concierge checks the branch you're merging into to see if your name is in the contributors.md, and decides based on that whether you're a new contributor.

But I think what happened here, is that since Concierge checks after the PR is merged, then your name is already in contributors.md in master, so it doesn't post. I think I need to check for the commit when the PR was opened, not when the PR is merged.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants