BioJulia / BioAlignments.jl

Sequence alignment tools
MIT License
60 stars 24 forks source link

Prepare for minor release #50

Closed alyst closed 3 years ago

alyst commented 3 years ago

It would be nice to tag a new minor release (2.1.0?), so I've done some preparatory work:

alyst commented 3 years ago

@jakobnissen can you please look at it?

jakobnissen commented 3 years ago

It all looks good to me, I agree with this. Do you know why AppVeyor is even run here? Does it need to be disabled somewhere external? I'll look into it.

alyst commented 3 years ago

Do you know why AppVeyor is even run here? Does it need to be disabled somewhere external? I'll look into it.

I cannot check this repository settings, but it could be it's somewhere in "integrations" or "webhooks"

alyst commented 3 years ago

@jakobnissen I've also noticed that documenter deploy is failing (you can see it in github actions status for the master head). But I think it's outside of ci.yml, maybe you can check it as well.

kescobo commented 3 years ago

Do you know why AppVeyor is even run here?

AppVeyor was used in a lot of repos to test on Windows, since Travis didn't do that. Maybe it's an artifact of a previous testing setup, but since github actions, can do everything, maybe it's not needed anymore?

alyst commented 3 years ago

Do you know why AppVeyor is even run here?

... but since github actions, can do everything, maybe it's not needed anymore?

I'd say definitely it has to be removed, since now windows testing is done by github actions. I was updating BioAlignments for github actions, but since I'm not an admin here, I couldn't disable appveyor.

jakobnissen commented 3 years ago

Done, the webhook to Travis and AppVeyor is deleted.

alyst commented 3 years ago

@jakobnissen thanks! are there also some traces of documenter deployment config that could be updated?