Deploy skripty #63

Merged
vasek merged 1 commits from mj-deploy into master 3 years ago
vasek commented 3 years ago

skripty instalující grafík na Gimliho

skripty instalující grafík na Gimliho
vasek added this to the beta milestone 3 years ago
mj commented 3 years ago

Ve větvi jsem uklidil, za mně ready to merge.

Ve větvi jsem uklidil, za mně ready to merge.

Vyzkoušel jsem a podle mne funguje. Díky. 👍

Jediné co, tak si nejsem jistý, jestli by nebylo lepší mít tohle někde v privátním repozitáři. Lidem venku to může být jedno, jak tohle funguje a u nás by se tak třeba dal kód toho skriptu deduplikovat. Takhle když se změní třeba jméno zámku, tak to začne být nekonzistentní s normálním buildem webu.

Zatím bych to ale neřešil. Můžeme to kdykoliv změnit. Merguji.

Vyzkoušel jsem a podle mne funguje. Díky. :thumbsup: Jediné co, tak si nejsem jistý, jestli by nebylo lepší mít tohle někde v privátním repozitáři. Lidem venku to může být jedno, jak tohle funguje a u nás by se tak třeba dal kód toho skriptu deduplikovat. Takhle když se změní třeba jméno zámku, tak to začne být nekonzistentní s normálním buildem webu. Zatím bych to ale neřešil. Můžeme to kdykoliv změnit. Merguji.
vasek merged commit 103e471436 into master 3 years ago
mj commented 3 years ago

Také bych zatím neřešil. V deploy skriptech jsou komentáře upozorňující na další uživatele zámku, tak snad zmatek nehrozí.

Také bych zatím neřešil. V deploy skriptech jsou komentáře upozorňující na další uživatele zámku, tak snad zmatek nehrozí.
This repo is archived. You cannot comment on pull requests.
No reviewers
No Milestone
No Assignees
2 Participants
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.