Skip to content
This repository was archived by the owner on Jul 18, 2022. It is now read-only.

Add Svelte CI/CD#105

Merged
cofob merged 5 commits intocofob/sveltefrom
cofob/add-npm-cicd
Apr 28, 2022
Merged

Add Svelte CI/CD#105
cofob merged 5 commits intocofob/sveltefrom
cofob/add-npm-cicd

Conversation

@cofob
Copy link
Copy Markdown
Contributor

@cofob cofob commented Apr 28, 2022

Depends on #99

@cofob cofob requested a review from PerchunPak April 28, 2022 15:14
@PerchunPak
Copy link
Copy Markdown
Contributor

PerchunPak commented Apr 28, 2022

Подожди, ты же просто запускаешь npm build и даже не коммитишь потому что все что скомпилированое в .gitignore.

Вообще предлагаю это засунуть в deploy скрипт и запускать на тэг. PyPI пуш тоже кстати не нужен, тк мы изменили концепцию.

В общем не понял смысл и что этот ревью должен делать. Давай обсудим и сделаем его вместе после мерджа #99.

@PerchunPak PerchunPak closed this Apr 28, 2022
@PerchunPak PerchunPak deleted the cofob/add-npm-cicd branch April 28, 2022 15:21
@cofob cofob restored the cofob/add-npm-cicd branch April 28, 2022 15:21
@cofob cofob reopened this Apr 28, 2022
@cofob
Copy link
Copy Markdown
Contributor Author

cofob commented Apr 28, 2022

Не так быстро. Это тест работоспособности Svelte кода. Если сборка не проходит - он сломан.

@cofob
Copy link
Copy Markdown
Contributor Author

cofob commented Apr 28, 2022

В общем не понял смысл и что этот ревью должен делать. Давай обсудим и сделаем его вместе после мерджа #99.

Я в описании указал что он зависит от #99

@cofob cofob changed the base branch from master to cofob/svelte April 28, 2022 15:23
@PerchunPak
Copy link
Copy Markdown
Contributor

PerchunPak commented Apr 28, 2022

В общем не понял смысл и что этот ревью должен делать. Давай обсудим и сделаем его вместе после мерджа #99.

Я в описании указал что он зависит от #99

Таким образом сделав Merge-Hell. Добавь это в #99 вместо отдельного PR. Да и я уверен есть что то по типу npm test без компилирования кода, что будет намного быстрее.

@cofob
Copy link
Copy Markdown
Contributor Author

cofob commented Apr 28, 2022

Да и я уверен есть что то по типу npm test без компилирования кода, что будет намного быстрее.

Тут нет npm test, потому что тестов - нет. Компиляция происходит довольно шустро.

@cofob cofob merged commit 6c7a716 into cofob/svelte Apr 28, 2022
@cofob cofob deleted the cofob/add-npm-cicd branch April 28, 2022 15:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@PerchunPak PerchunPak Awaiting requested review from PerchunPak

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cofob @PerchunPak