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

DaniaB24: tasks submissions in other repos (3 tasks pending) #174

Open
4 tasks done
DaniaB24 opened this issue Aug 16, 2022 · 7 comments
Open
4 tasks done

DaniaB24: tasks submissions in other repos (3 tasks pending) #174

DaniaB24 opened this issue Aug 16, 2022 · 7 comments

Comments

@DaniaB24
Copy link
Contributor

DaniaB24 commented Aug 16, 2022

@DaniaB24 DaniaB24 changed the title github-DaniaB24 github-DaniaB24 prs-2019-p2p Aug 16, 2022
@OleksiyRudenko OleksiyRudenko changed the title github-DaniaB24 prs-2019-p2p DaniaB24: prs-2019-p2p Aug 16, 2022
@OleksiyRudenko
Copy link
Member

Thank you for opening this.

As I can see, neither of the linked PRs is merged.
Some contain review feedback that is not addressed yet.
And this one contains multiple files not related to the task per se.

My proposal:

  1. Improve your code as per reviewers' feedback.
  2. Resubmit listed tasks in this repo.
  3. Make sure each PR contains only relevant and required files/commits as per this repo submission instructions.

Let me know if you have any questions.

@OleksiyRudenko OleksiyRudenko changed the title DaniaB24: prs-2019-p2p DaniaB24: prs-2019-p2p (void) Aug 16, 2022
@OleksiyRudenko
Copy link
Member

Closing as listed PRs are not approved and cannot be accounted for.

@OleksiyRudenko
Copy link
Member

Re-opening to have visibility on progress across other homeworks repos.

@OleksiyRudenko OleksiyRudenko changed the title DaniaB24: prs-2019-p2p (void) DaniaB24: prs-2019-p2p (completion pending) Aug 22, 2022
@OleksiyRudenko OleksiyRudenko changed the title DaniaB24: prs-2019-p2p (completion pending) DaniaB24: tasks submissions in other repos (completion pending) Aug 22, 2022
@OleksiyRudenko OleksiyRudenko changed the title DaniaB24: tasks submissions in other repos (completion pending) DaniaB24: tasks submissions in other repos (3 tasks pending) Aug 22, 2022
@stale
Copy link

stale bot commented Sep 21, 2022

discarded

@stale stale bot added the 💤 Stale label Sep 21, 2022
@OleksiyRudenko
Copy link
Member

Up

@stale
Copy link

stale bot commented Oct 12, 2022

discarded

@stale
Copy link

stale bot commented Apr 12, 2023

This issue has been automatically marked as stale because there were no activity during last 14 days. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

А. Чому так?
Найбільш розповсюджена причина: Студент не реагує на коментарі змінами коду і не задає запитань через брак часу або зміну життєвих пріоритетів. Покинуті піари відволікають менторів. Коли у студента з'явиться час, він/вона зможе перевідкрити той самий піар і продовжити роботу.

Б. Що робити, якщо в піарі нема оновлень, оскільки не зрозуміло, що треба зробити?
Варіант 1. Задати питання в самому PR.
Варіант 2. Задати питання в студентському чаті.

В. А якщо я все зробив(ла) і це ментор не рев'юває мої зміни?

  1. Переконайся, що ти відреагував(ла) на всі коментарі або кодом, або запитаннями, або відповідями. Напиши в PR і в чаті, що чесно вважаєш, що все зроблено і попроси повторне рев'ю. Якщо щось не зрозуміло, задай запитання.
  2. Реагуй на коментарі як менторів, так і інших учасників, включаючи ботів.
  3. Не ігноруй прохання типу * "Let's do some self-checks ..." * "Go through the checklist below..." * "mark fulfilled requirements..." * "if you would silently ignore this recommendation, a mentor may think that you are still working on fixes"
    навіть якщо вони написані ботом. Боти помічники і ментори покладаються на те, що прохання і пропозиції бота дотримуються.
    Не лінись піти по лінках в коментарях, погуглити термінологію та скористатись Google Translate.
  4. Можливо, у менторів склалися інші пріоритети через роботу, сімейні обставини і т.п. В такому разі, якщо ти зробив(ла) рекомендоване вище, то волай в чаті, що PR позначений stale, наче, все зроблено, а ментори чомусь не реагують - рятуйте!

Г. Хіба недостатньо того, що я додав(ла) коміт із змінами?
Часто буває так, що бачиш новий коміт, ідеш перевіряти, змін багато, доводиться перечитувати весь код. А потім з'ясовується, що одна невеличка зміна "відкладена на потім" чи з'являється ще один коміт і знов треба перечитувати все. Любіть нас, спілкуйтеся з нами - і ми відповімо повною взаємністю.

Традиційна пропозиція: задай питання по вищенаписаному в студентському чаті.

@stale stale bot added the 💤 Stale label Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants