search

kottans / frontend-2022-homeworks

15 stars 185 forks source link

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

Open DaniaB24 opened 2 years ago

DaniaB24 commented 2 years ago
OleksiyRudenko commented 2 years ago

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 commented 2 years ago

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

OleksiyRudenko commented 2 years ago

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

stale[bot] commented 1 year ago

discarded

OleksiyRudenko commented 1 year ago

Up

stale[bot] commented 1 year ago

discarded

stale[bot] commented 1 year ago

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, наче, все зроблено, а ментори чомусь не реагують - рятуйте!

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

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