Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Změna deploye na jinou větev (např. production) #21

Open
3 tasks done
lukaszach opened this issue Aug 2, 2019 · 3 comments
Open
3 tasks done

Změna deploye na jinou větev (např. production) #21

lukaszach opened this issue Aug 2, 2019 · 3 comments
Assignees

Comments

@lukaszach
Copy link

lukaszach commented Aug 2, 2019

Vzhledem k tomu, že master je středobod repository - běžně je to "uzel", do kterého proudí všechny MR, tak není vhodné, aby se ten vždy vydával na produkci.

Vhodný způsob pro continuous integration je použití jiné větve, do které se master merguje a pomocí CI dojde teprve tehdy k releasu.

Navrhuji vytvořit novou větev production, do které se v případě potřeby bude mergovat master. Následně by došlo k releasu.

Výhody:

  • souběžný vývoj
  • možnost prereleasu (testing/staging)
  • stále aktuální master

Postup:

  • upravit aktuální CI na větev production
  • vytvořit větev production

Současně:

  • v CI změnit npm za yarn

Motivace: Aplikace neobsahuje ani yarn.lock, ani package-lock.json. Vhodné je zvolit jednu variantu (asi yarn.lock) jako součást repository a podle ní pak instalovat závislosti. Tím vzniká vyšší pravděpodobnost i jistota, že se něco při drobné úpravě nerozbije na závislostech.

@lukaszach lukaszach self-assigned this Aug 2, 2019
@lukaszach
Copy link
Author

@horm dává ti to smysl?

@HormCodes
Copy link
Contributor

@lukaszach jojo souhlas, předpokládám, že to takto provedeme i u ostatních repozitářů, že?

@lukaszach
Copy link
Author

@horm ano, zde bych asi začal, protože to je docela podstatné a pak zkusil jinde. Tím by se dalo vytvořit i testovací prostředí s vlastní URL - osobně nevím pricing za takovou další instanci. Dalo by se to domyslet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants