Hexlet / hexlet-correction

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

[#255] Strip out unnecessary elements from the page #257

Closed kitdim closed 2 months ago

kitdim commented 2 months ago

Поправил. image

bazilval commented 2 months ago

Отдельные изменения стоит выносить на pull request отдельными ветками Советую глянуть вот этот гайд: https://ru.hexlet.io/blog/posts/Open-Source-github

kitdim commented 2 months ago

@bazilval ок, а это разве не из отдельной ветки? image

bazilval commented 2 months ago

@kitdim действительно, отдельной просто видно, что вы подтягиваете коммиты, которые относятся к другому пулл-реквесту по-хорошему они должны быть полностью независимы друг от друга image

fey commented 2 months ago

Вижу другой ПР со стилем. Его я смержил, поэтому этот лучше отребейзте - удалите просто лишний коммит и сделайте форс пуш.

fey commented 2 months ago

@kitdim у вас все еще лишний коммит, изменяется другой файл. Должен ли он изменяться? Попробуйте ребейзнуть ветку, удалить лишние коммиты.

kitdim commented 2 months ago

@fey Привет! Ок, поправлю)

kitdim commented 2 months ago

@fey готово, посмотри как будет время