search

aminvakil / docker-ocserv

OpenConnect VPN Server Docker
https://quay.io/aminvakil/ocserv
82 stars 35 forks source link

Feature/ocserv version autoupdate #62

Closed framebassman closed 5 months ago

framebassman commented 5 months ago

https://github.com/aminvakil/docker-ocserv/settings/actions image

aminvakil commented 5 months ago

This is also getting checked in https://github.com/aminvakil/nvchecker/blob/main/others.toml, but it would not open a PR in this repository yet like docker images, so I think this is useful.

But I still prefer to have this in nvchecker repository, could you please open a PR there? It could not be generic like docker_other_repos workflow in nvchecker though, it can be just like this but opening a PR in this repository from nvchecker repository.

If you do not have time for it, I will review and merge this PR in this repository until I find some time to migrate this to nvchecker repository and then revert this.

framebassman commented 5 months ago

@aminvakil I suggest to merge this one. After that I will create the PR in nvchecker (it may take some time because it is not so obvious) and we will revert this one

aminvakil commented 5 months ago

Please do not force-push on every commit, it makes reviewing harder, we will squash and fix commits at last.

aminvakil commented 5 months ago

Thank you for doing this!

I will open a PR whenever I found some time (and if you didn't beat me to it) in https://github.com/aminvakil/nvchecker later on, so this bumping would be centralized and could be used for other stuff as well.

Then this will be reverted.

But until then, this would definitely be an improvement, thanks again!