LINUX.ORG.RU

github не проходит автоматическая проверка реквеста

 


0

0

Как правильно поступить в случае

All checks have failed

1 failing check

continuous-integration/travis-ci/pr — The Travis CI build failed

Отменить реквест, исправить и сделать новый реквест или закоммитить исправление в ветку?

★★★★★

как хочешь, но быстрее будет закоммитить в ветку.

cetjs2 ★★★★★
()

Пофиксить в том же реквесте.

Когда владелец решит, что можно мёржить, сквашишь коммиты. Обычно сами просят подчистить историю.

Отменять на мой взгляд можно только в том случае, когда реквест по каким-либо причинам не может быть принят вообще, иначе скапливается мусор из «недоделанных» реквестов.

Kilte ★★★★★
()
Последнее исправление: Kilte (всего исправлений: 1)

Самое правильное - сделать git commit --amend (или rebase) + push --force.

slovazap ★★★★★
()

Есть мнение, что конкретно на Github лучше просто коммитить исправления в ветку без всяких rebase впоследствии, так как единицей изменения кодовой базы является пулл-реквест целиком, а не отдельный коммит.

ilammy ★★★
()
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.