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

Use generic travis install script rather than OS specific ones #1408

Merged
merged 1 commit into from
Jun 17, 2017

Conversation

bsipocz
Copy link
Contributor

@bsipocz bsipocz commented Jun 17, 2017

This should address the issue @kain88-de mentioned about the cron builds in astropy/ci-helpers#199 (comment)

Copy link
Member

@richardjgowers richardjgowers left a comment

Choose a reason for hiding this comment

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

Lgtm assuming it works

@kain88-de kain88-de merged commit 9dc1737 into MDAnalysis:develop Jun 17, 2017
@kain88-de
Copy link
Member

@bsipocz Thanks

@orbeckst orbeckst mentioned this pull request Jun 21, 2017
IAlibay added a commit that referenced this pull request Dec 10, 2023
tylerjereddy pushed a commit that referenced this pull request Dec 15, 2023
* Add bsipocz from PR #1408

* David van der Spoel from commit 54ed4b3

* gh seems to indicate a 2012 merge (commit may have been circa 2010?)

* Add mimischi from PR 2356

* Fix date for 2.6.1 and duplicate authors in changelog
ljwoods2 pushed a commit to ljwoods2/mdanalysis that referenced this pull request Dec 15, 2023
* Add bsipocz from PR MDAnalysis#1408

* David van der Spoel from commit 54ed4b3

* gh seems to indicate a 2012 merge (commit may have been circa 2010?)

* Add mimischi from PR 2356

* Fix date for 2.6.1 and duplicate authors in changelog
orbeckst added a commit that referenced this pull request Dec 21, 2023
Update of AUTHORS and CHANGELOG with inferred author contributions.

*  Removed duplicate mattwthompson in 0.20.0 changelog entry.: mattwthompson was placed twice by accident, this removes this duplication.

* Addition of missing authors.

   An retrospective analysis of the authors found via `git shortlog -s -n --email --branches="develop"` found several commits by authors which were not present in the `AUTHORS.md` file.

    - Zhenbo Li: commited via PR: Started tests for gnm. #803 and Make Travis run tests on OSX. #771,
    - Jenna M. Swarthout via PR Update CoC according to suggestions from current CoC committee #4289 and Point to new reporting form link (owned by [email protected]) #4298,
    - Bradley Dice via PR   Fix GSD Windows compatibility #2384 ,
    - David Minh via PR #2668

   There seemed to be no indications in the above mentioned PRs that the author did not want to be in the authors file, it looks like it was just forgotten.

* Addition of missing entries from the changelog

   Continued cross referencing of the git shortlog output but also accounting for the changelog identified several individuals that had not been included in the changelog entries for the release they contributed under. They were added to the relevant entry of the changelog based on the merge commit date.

   Please note that for Tone Bengsten, we a) had no github handle (so they were assigned @tbengtsen), and b) no specific commit. Given we know that this individual was including alongside the encore merge, they were assigned to the 0.16.0 release.

* Update CHANGELOG
* PR #1218
* PR #1284 and #1408
* PR #4109
* based on git history
* PRs #803 and #771 (author addition, changelog addition)
* PR #2255 and #2221
* PR #1225
* PR #4289 and #4298
* PR #4031
* PR #4085
* PR #3635
* PR #2356
* PR #2559
* No GH handle - Encore author addition @tbengtsen
* PR #4184
* PR #2614
* PR #2219
* PR #2384
* PR #2668
* Add missing entry for Jenna

Thanks to @fiona-naughton for helping out with digging into this data :)

Co-authored-by: Fiona Naughton <[email protected]>
Co-authored-by: Oliver Beckstein <[email protected]>
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.

3 participants