search

Hexlet / hexlet-correction

Typo Reporter
https://fixit.hexlet.io/
GNU Affero General Public License v3.0
41 stars 72 forks source link

[#177] rename TypoEvent constants to change buttons names #239

Closed nuuska-muikkunen closed 3 months ago

Malcom1986 commented 3 months ago

@nuuska-muikkunen Борис, предлагаю и сами ивенты в коде тоже не трогать, они на мой взгляд неплохо названы, по аналогии с ишьюсами в гитхабе. Достаточно просто надпись на кнопке поменять. Также если докерфайл поправите и задеплоите демку на render, будет круто! Чтобы можно было потыкать, проверить как выглядит и работает

nuuska-muikkunen commented 3 months ago

Удивляюсь, как это все раньше работало :-( Пытался деплоить:

  1. gradle installDist не прошел - нет такой задачи в проекте
  2. Добавил плагин distribution - получил task installDist
  3. Добился завершения сборки через build с выключенными тестами (тесты требовали, что-то от докера и падали)
  4. Теперь приложение не нашло манифест - указал Main-Class
  5. Теперь не узнает этот класс - HexletTypoReporter с ошибкой "java.lang.NoClassDefFoundError: org/springframework/core/env/Environment"
  6. Обновил версии liquibase и webjars-locator (они не соответствовали версии спринга) Так и не помогло ((( Может я не в ту сторону копаю и надо было изменить запуск приложения, а не создание jar файла? Буду возвращаться к ивентам и кнопкам...
nuuska-muikkunen commented 3 months ago

К сожалению, в таком варианте не хватает памяти на бесплатном тарифе рендера. Gradle build daemon disappeared unexpectedly (it may have been killed or may have crashed)

А так все кнопочки работают. Может есть где еще попробовать деплоить?

nuuska-muikkunen commented 3 months ago

Или подскажите, поможет ли плагин application создать директорию build/install с jar файлом? distribution не помог (

fey commented 3 months ago

Я в демке создал воркспейс, но мне совершенно некуда было снипет прикрутить, чтобы опечатку создать)) Я думаю, что тут норм все, ведь взиуально только кнопка переименовывается. По коду @Malcom1986 попросим ревью

nuuska-muikkunen commented 3 months ago

life hack Для снипета я использовал сайт - мою персональную страничку на GitHub'е, который мы на курсе делали. Там один html, который я могу редактировать, как хочу. .github.io

nuuska-muikkunen commented 3 months ago

@Malcom1986 Максим, вдруг увидел, что просили ивенты не трогать. Исправил. Теперь только имена кнопок изменены. Docker запускает gradle, комментарии со старыми командами maven убрал. Prod для проверки по адресу https://hexlet-correction-rqc5.onrender.com/

Malcom1986 commented 3 months ago

Нормально завелось на рендере с новым докерфайлом?

nuuska-muikkunen commented 3 months ago

Да, заработало. Указал параметр среды, как в инструкции рендера - у него только такая работает. Ну и по инструкции из докера понял что такое тег :-) Сразу заработало. Ну и удалил лишнюю строчку из README на последок. Все готово. P.S. С понедельника выхожу на работу. Буду медленнее реагировать.