Skip to content

Lesson 3#423

Open
Sergei-Krivonosov wants to merge 8 commits intomainfrom
lesson_3
Open

Lesson 3#423
Sergei-Krivonosov wants to merge 8 commits intomainfrom
lesson_3

Conversation

@Sergei-Krivonosov
Copy link
Copy Markdown
Collaborator

No description provided.

@Sergei-Krivonosov Sergei-Krivonosov linked an issue Apr 25, 2026 that may be closed by this pull request
@chumax111
Copy link
Copy Markdown
Collaborator

Привет, нужно конфликты исправить, и не забывай в коммитах писать что ты делаешь ( а не 1, 2, 3), иначе непонятны изменения ревьюеру

@Sergei-Krivonosov
Copy link
Copy Markdown
Collaborator Author

Пока я жду по два дня апрувов, происходят изменения, я уже третий раз после всех исправлений разрешаю конфликты.

Comment thread tests/test_folder.py

browser.find_element(By.ID, "name").send_keys(FOLDER_NAME)
browser.find_element(By.CLASS_NAME, "com_cloudbees_hudson_plugins_folder_Folder").click()
browser.find_element(By.ID, "ok-button").click()
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ты при решении конфликта откатил на старую версию функцию?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нет. Я просто удалил данные своего теста

Copy link
Copy Markdown
Collaborator

@StasAxlS StasAxlS Apr 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ты что то исправляешь, удаляешь со всеми тестами в test_folder.py.
И говоришь "Нет. Я просто удалил данные своего теста". Я просто не понимаю

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я удалил импорт, который был нужен только моему тесту. Я удалил свой тест.
Создал файл фолдер_2, все это перенес туда. запушил. Все.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

То что ты говоришь не соответствует твоему PR.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ничего больше не делал. Только то, что написал.

@Sergei-Krivonosov
Copy link
Copy Markdown
Collaborator Author

Начиная с субботы, когда я прошел проверки CI не было никаких конфликтов. Они появляются через 6-8 часов, пока я жду ревью. ни кто не проводит. Появляются конфликты. Я и два раза разрешил. Все также тишина. Сейчас решил не дополнять файлы, с которыми есть конфликты, создал свой, свои данные из других файлов удалил. Никаких конфликтов сейчас нет.

@StasAxlS
Copy link
Copy Markdown
Collaborator

Попроси в qa_python "Видео как решать конфликты". Вроде было раньше видео с объяснениями. Конфликты всегда будут.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TC_01.003.05 | Create new folder from copy.

3 participants