Что делать, если ваш коммит не прошел проверку GitHub?

Question

Хорошо, я только что узнал о модульном тестировании, GitHub проверяет CircleCI и выполняет запросы на вытягивание всего за день. У меня возник вопрос, когда я пытался проверить свое понимание всего:

1. У меня есть локальный репозиторий, и он настроен на pu sh изменения моего репо на GitHub.
2. У меня есть несколько базовых c тестов, которые проходят pytest тестовый запуск локально.
3. Я делаю простое изменение, чтобы все проверить, фиксирую и sh изменение.
4. Я go в GitHub и вижу, что я не прошел тесты, определенные в моем YAML-файле CircleCI, потому что в моей виртуальной среде не установлен flake8 (я случайно установил его глобально).
5. Я возвращаюсь в PyCharm и устанавливаю flake8 в свой venv.
6. Я обновляю requirements.txt, чтобы отразить установку.
7. ???

Теперь у меня такой вопрос: если проверка не удалась, как лучше всего «обновить» пулреквест после исправления проблемы. Могу ли я просто сделать новую фиксацию, которая исправляет это, и начать новый запрос на перенос? Или я найду способ отменить старую фиксацию, чтобы включить новую, которая работает?

Answer 1

Запрос на вытягивание - это черновик ветки. До того, как он будет объединен с целевой веткой, он на самом деле не опубликован. Вы можете делать все, что хотите, до тех пор, пока его окончательная версия соответствует ожиданиям. Его история коммитов может выглядеть некрасиво, но в нем хранится запись о том, что вы сделали для цели. Люди могут предпочесть более красивую историю, даже если это черновик. В этом случае можно использовать git rebase -i, git reset --hard, git commit --amend или даже git filter-branch, чтобы переписать аккуратную и красивую историю, а затем использовать git push -f для обновления запроса на перенос. Конечно, если другие также работают над одним и тем же запросом на перенос, всегда уведомляйте их, прежде чем вы захотите принудительно обновить запрос на перенос. С другой стороны, если он сква sh слит с целевой ветвью, его история, неважно, некрасивая она или красивая, не будет передана в целевую ветку.