MobleyLab / chemper

Repository for Chemical Perception Sampling Tools
MIT License
19 stars 10 forks source link

Migrate from Travis CI to GitHub Actions #102

Closed mattwthompson closed 2 years ago

mattwthompson commented 2 years ago

Not intended to be a part of this PR

codecov[bot] commented 2 years ago

Codecov Report

Merging #102 (e652165) into master (cc82c7d) will decrease coverage by 6.37%. The diff coverage is 100.00%.

mattwthompson commented 2 years ago

If nothing else, the status passing in the last commit (merging #101) shows that that change passes tests.

mattwthompson commented 2 years ago

I've taken the liberty of also removing un-used scripts (largely boilerplate from the cookiecutter) and updating some of the outdated portions of the README.

mattwthompson commented 2 years ago

The phantom Travis CI status check is blocking my ability to merge since this is in @MobleyLab, not @openforcefield. Are you able to override the status check requirement as an administrator @j-wags? If not we'll need to ping David and/or Caitlin.

davidlmobley commented 2 years ago

@mattwthompson can I do something to help?

mattwthompson commented 2 years ago

Could you merge? This is what I see:

image

but you may have the "click here to use your administrator privileges" or something to that effect.

This is because the Travis CI check is being reported as required but is removed here. It'll probably not be an issue moving forward. I deleted the webhook, too, for good measure.

davidlmobley commented 2 years ago

Very weird, it's showing me the same thing. I also have not yet found the place to disable the thing which requires status checks to pass before merging. :( Investigating.

davidlmobley commented 2 years ago

Wait, NOW it works -- I may have had to re-authenticate on GitHub. Merging.

mattwthompson commented 2 years ago

Found it - there was a separate branch protection rule requiring that Travis CI pass (not simply a "make sure they're all green, no matter what they are"), which I manually removed.

mattwthompson commented 2 years ago

All done - thanks!