[#272] fix the button "Hexlet Type Reporter"

Hexlet / hexlet-correction

Typo Reporter

https://fixit.hexlet.io/

GNU Affero General Public License v3.0

41 stars 72 forks source link

[#272] fix the button "Hexlet Type Reporter" #273

Closed kitdim closed 3 weeks ago

HelenOne commented 4 weeks ago

@kitdim тут (https://github.com/Hexlet/hexlet-correction/pull/270) вроде то же самое?

kitdim commented 4 weeks ago

@HelenOne неа, тут вот эта 272 в той убирали уникальность поля, а тут кнопка не работала. я похоже коммиты объединил, думаю не критично

bazilval commented 4 weeks ago

@kitdim а можно демо?

по поводу коммитов: чтобы было опрятнее, желательно отдельные фичи делать в отдельных ветках и чтобы они не наследовались друг от друга, так как твою другую фичу могут не вмержить, а эту захотят, например

поэтому полезно делать их независимыми кирпичиками

kitdim commented 4 weeks ago

@bazilval вот демо

kitdim commented 3 weeks ago

@fey привет! Тут вроде закончили, можно мержить

fey commented 3 weeks ago

А я демку еще не посмотрел :\ Чет не открывается на рендере.

По поводу коммитов да. на будущее лучше воздерживаться от мерж коммитов, которые происходят из-за пулла. тут расписано, как этого избежать: https://ru.hexlet.io/qna/git/questions/kak-aktualizirovat-vetku-v-pull-request

fey commented 3 weeks ago

Пока смержу, деплой у нас все равно не работает (сломано из-за др ПРа)

kitdim commented 3 weeks ago

да, долго что-то грузит, ща вот так выглядит