donaldzou / WGDashboard

Simple dashboard for WireGuard VPN written in Python & Vue.js
https://donaldzou.github.io/WGDashboard-Documentation/
Apache License 2.0
1.65k stars 254 forks source link

Workflow change #447

Closed DaanSelen closed 3 weeks ago

DaanSelen commented 4 weeks ago

This changes the Docker image building process from every night to whenever a commit is issued to the main branch. Ensuring the container is not unnecessarily built.

ANALYZING will stay every 24HRs because a CVE can pop-up every day.

DaanSelen commented 4 weeks ago

I screwed this PR.

FIRST REVIEW https://github.com/donaldzou/WGDashboard/pull/437

donaldzou commented 3 weeks ago

Hi Daan, do you mean the previous PR have incorrect build setup for Docker?

DaanSelen commented 3 weeks ago

No, its the same as the previously mentioned PR, but I accidentally submitted the changes twice. https://github.com/donaldzou/WGDashboard/pull/447/commits/a93291b38fd1219586c06fafa86ee87916621a4a

This is the only relevant change in this PR

donaldzou commented 3 weeks ago

Ah gotcha. Lmk when is good to merge ;)

DaanSelen commented 3 weeks ago

If you approve the https://github.com/donaldzou/WGDashboard/pull/437 (feel free to merge at your own will).

Then you can look at this which is basically just a schedule change.

DaanSelen commented 3 weeks ago

Thanks!