Natixar / Blockchain-frontend

Other
0 stars 0 forks source link

Deployment Script is NOT Configured for Manual Running #4

Open lepeuvedic-natixar opened 1 month ago

lepeuvedic-natixar commented 1 month ago

Problem

https://github.com/Natixar/Blockchain-frontend/blob/cb0d07e4cb5f62c82b7418b30cecd8560df6a5b1/.github/workflows/deploy.yml#L4 There is no way to deploy the current version to a new server.

How to Reproduce?

Go to Actions and there is no option to deploy.

What is Needed Instead?

Automatic deploy on pushes to the main branch should only be used when direct pushes to the main branch are disabled, and only allowed after a thorough review via a pull request. It is not acceptable to let the server-hosted version update when anyone pushes anything, without a prior review and sufficient quality control (like checking that the unit tests actually run).

Manual deploys must be allowed to install the main branch version to a new testing environment too.

Sagerahl commented 1 month ago

Update on GitHub Actions (see e08cd55):

Branch Protection and Development Branch: