Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix fetchdata.py issues fetch #369

Merged
merged 1 commit into from
Nov 29, 2023
Merged

Conversation

DREU007
Copy link
Contributor

@DREU007 DREU007 commented Nov 28, 2023

Пофиксил некорректную обработку issues во время подкачки с GitHub.

Проблема:
При передачи дефолтного значения type_='iss' в create_contributions(), оно перезаписывалось как 'pr' при первом совпадении и дальше на каждом цикле не верно проверяло и приписывало issues к pull requests.

Решение:
Введена новая переменная true_type которая хранит данные для текущего сontibution, а дефолтная type_ не переиспользуется.

@fey
Copy link
Collaborator

fey commented Nov 29, 2023

У вас в пулл реквесте есть изменения из другого ПРа, который еще не посмотрели. https://github.com/Hexlet/hexlet-friends/pull/368/files
Можете пожалуйста сделать ребейз и удалить лишние коммиты, чтобы оставались только изменения по текущей задачке?

@fey
Copy link
Collaborator

fey commented Nov 29, 2023

плюсом вижу парочку мерж реквестов. Предлагаю их тоже убрать.
Если нужно обновлять свою ветку, то изменения лучше подтягивать командой git pull --rebase upstream main

@DREU007
Copy link
Contributor Author

DREU007 commented Nov 29, 2023

Перезалил ветку.

@fey fey requested a review from sgmdlt November 29, 2023 12:44
@sgmdlt sgmdlt merged commit eb07d6d into Hexlet:main Nov 29, 2023
1 check passed
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.

3 participants