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

Rewrite #35

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

Rewrite #35

wants to merge 22 commits into from

Conversation

eolme
Copy link

@eolme eolme commented May 16, 2021

  • Добавил eslint и конфиг VKCOM

    • Выключен guard-for-in потому что гарантируется чистые объекты
    • Выключен comma-dangle чтобы гарантировать поддержку nodejs
  • Версия nodejs поднята до минимально поддерживаемой

  • Убрано множество пакетов

  • Переписан код полинга событий для более быстрой и правильной обработки

  • Добавлены проверки ответов от API и соответствующие обработки ошибок

  • Добавлены проверки на доступность для чтения файлов

  • Полная поддержка CI/CD

  • Добавлено управление параметрами через vk-hosting-config.json

Closes: #9
Closes: #29

@eolme
Copy link
Author

eolme commented May 16, 2021

@Mobyman @kuhel

@eolme
Copy link
Author

eolme commented Aug 9, 2021

@lisovskyR @ilyapishchulin может вы подскажете?

@mainsmirnov
Copy link
Contributor

Хай, спасибо, посмотрим

@mainsmirnov
Copy link
Contributor

Хай, стянул твои изменения, пытаюсь через yarn deploy отправить файл и валится с ошибкой от апи
ERR_UNKNOWN: save bundle result "0"Bad SVG file

Переключился на мастер и там всё ок

@eolme
Copy link
Author

eolme commented Aug 9, 2021

Хорошо, гляну, спасибо. Можешь скинуть на чем потестить? У меня все ок

@lisovskyR
Copy link

build.zip
vk-hosting-config.json.zip
я пробовал на этих данных

@eolme
Copy link
Author

eolme commented Aug 10, 2021

Проблема в пакете zip-a-folder, откатил на старую.
Добавил ссылку на Troubleshooting, дополнил документацию.

package.json Outdated Show resolved Hide resolved
@mainsmirnov
Copy link
Contributor

Мне всё нравится, но смогу потестить только в начале следующей недели

README.md Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Деплой только в прод или только в дев Поддержка CI
5 participants