search

galaxycomputationalchemistry / galaxy-tools-compchem

:mega: Galaxy Tools for Computational Chemistry
Apache License 2.0
14 stars 16 forks source link

Update tool versions #78

Closed simonbray closed 4 years ago

bgruening commented 4 years ago

Wonderful! If we are happy with github actions, we should remove travis.

I'm wondering if you are already using the automatic version bump? :)

simonbray commented 4 years ago

Wonderful! If we are happy with github actions, we should remove travis.

Yes, fine with me.

I'm wondering if you are already using the automatic version bump? :)

No, not at the moment. :( We need to talk about this again.

Btw, do not merge yet, I realised there is a new version of ambertools and acpype which can be added.

simonbray commented 4 years ago

Happy to merge now. How do I stop/remove the travis check? I thought this was dealt with in Chris' PR.

chrisbarnettster commented 4 years ago

Happy to merge now. How do I stop/remove the travis check? I thought this was dealt with in Chris' PR.

I removed the integration but forgot the webhook - fixed now. Webhook deleted.

simonbray commented 4 years ago

I removed the integration but forgot the webhook - fixed now. Webhook deleted.

Thank you - but I can still see the travis job in the list of checks for some reason?

bgruening commented 4 years ago

I think this is still because it is on the master branch. I would not worry about it until it is still there in the next PRs following this one.

simonbray commented 4 years ago

Can you force-merge this one then?

simonbray commented 4 years ago

Great, thank you!