Create greetings.yml

[hmacdope]

Greet first time contributors.

Changes made in this Pull Request:

• Added a greeting asking people to introduce themselves and abide by COC.

[codecov]

Codecov Report

Merging #3609 (6c7c043) into develop (3d8c75c) will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #3609      +/-   ##
===========================================
+ Coverage    94.21%   94.23%   +0.01%     
===========================================
  Files          190      190              
  Lines        24725    24752      +27     
  Branches      3321     3329       +8     
===========================================
+ Hits         23295    23324      +29     
+ Misses        1382     1380       -2     
  Partials        48       48              

Impacted Files	Coverage Δ
package/MDAnalysis/coordinates/MOL2.py	97.63% <0.00%> (ø)
package/MDAnalysis/topology/MOL2Parser.py	99.12% <0.00%> (+0.22%)	⬆️
package/MDAnalysis/analysis/lineardensity.py	87.65% <0.00%> (+3.23%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3d8c75c...6c7c043. Read the comment docs.

[RMeli]

Since this gets triggered on new PRs, it should also tell people to add themselves to the AUTHORS list? Changes to AUTHORS are often (always?) missing and reviewers have to explain this each time.

Worth mentioning Discord as well?

[IAlibay]

I'll push change a bit later today, but this needs to be limited to the core MDA repo, don't want folks playing around with forks getting bothered with this.

[hmacdope]

Anyone feel free to push changes they want directly :)

[RMeli]

Suggested change

pr-message: 'Hello there first time contributor! Welcome to the MDAnalysis community! We ask that all contributors abide by our Code of Conduct (https://www.mdanalysis.org/pages/conduct/) and that first time contibutors introduce themselves on the developer mailing list (https://groups.google.com/g/mdnalysis-devel) so we can get to know you.'

pr-message: 'Hello there first time contributor! Welcome to the MDAnalysis community! We ask that all contributors abide by our [Code of Conduct](https://www.mdanalysis.org/pages/conduct/) and that first time contributors introduce themselves on the [developer mailing list](https://groups.google.com/g/mdnalysis-devel) and the [MDAnalysis Discord Server](https://discord.com/channels/807348386012987462/) so we can get to know you. Please also add yourself to `package/AUTHORS` as part of this PR.'

[orbeckst]

For the Discord server I wouldn't link directly because that link is useless if you haven't been invited. https://www.mdanalysis.org/#participating has the discord information.

[IAlibay]

I went ahead and directly made changes, sorry just was faster to do it.

I've explained my changes below.

[orbeckst]

Greeting message looks good to me. Thanks for setting it up.

(I commented on a comment regarding discord, just saying that we can't directly link to our server, instead I suggest to link to https://www.mdanalysis.org/#participating .)

[IAlibay]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[IAlibay]

I'm not sure why azp is being so flaky, I'll try to re-run again.

Re: pypi checks - I suspect setuptools' recent version change is probably at fault here... this is going to be a long Friday evening.

[IAlibay]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[orbeckst]

I like the idea of customized messages for new contributors.

However, does the message replace our standard template? That would not be good because I really want to encourage everyone to follow the structure in the template, including the "fixes" and checklist.

Note that actions/first-interaction also has a way to greet first time issue openers. Is this something we want to do??

[IAlibay]

I like the idea of customized messages for new contributors.

However, does the message replace our standard template? That would not be good because I really want to encourage everyone to follow the structure in the template, including the "fixes" and checklist.

This should just be a separate comment on the PR, it shouldn't affect the template.

Issue wise, I'd be reluctant at the moment. Sometimes I like to drop by an org, raise an issue about something but not necessarily be invested in that community. I'd prefer just let us handle the initial response.

[orbeckst]

Makes sense, I’m fine with PR-only message.

…

Am 4/8/22 um 22:37 schrieb Irfan Alibay ***@***.***>: Issue wise, I'd be reluctant at the moment. Sometimes I like to drop by an org, raise an issue about something but not necessarily be invested in that community. I'd prefer just let us handle the initial response.