search

OLees / Polinom

0 stars 0 forks source link

Учимся создавать pull request :) #2

Open PanovaElena opened 3 years ago

PanovaElena commented 3 years ago

@OLees, это важно, потому что у меня появляется возможность комментировать отдельные строчки кода. Итак:

1) Создаем репозиторий, указываем "добавить README", чтобы автоматически создался первый коммит. 2) Клонируем репозиторий. 3) Создаем пользовательскую ветку, переходим в нее (команда git checkout). 4) Пишем код, делаем коммиты в эту ветку. 5) Заливаем код на гитхаб. 6) Делаем pull request из вашей ветки в main/master/..., не мержим его и не закрываем, а в комментариях указываем @PanovaElena

В общем, ничего сложного, нужно только быть внимательным.

По коду: все нормально, только обычно функция получения указателя в итераторе называется просто get(), можно назвать getPtr() или как-нибудь еще, но getit() сбивает с толку.

Принято.

OLees commented 3 years ago

Да, я понял. Просто я заливал вечером, забыв изначально создать проект, поэтому пришлось создавать так. Репозиторий по таблицам уже создал)     С уважением, Даниил Никитин @.***    

Пятница, 23 апреля 2021, 1:40 +03:00 от Elena Panova @.***>:     @OLees , это важно, потому что у меня появляется возможность комментировать отдельные строчки кода. Итак:

  • Создаем репозиторий, указываем "добавить README", чтобы автоматически создался первый коммит.
  • Клонируем репозиторий.
  • Создаем пользовательскую ветку, переходим в нее (команда git checkout).
  • Пишем код, делаем коммиты в эту ветку.
  • Заливаем код на гитхаб.
  • Делаем pull request из вашей ветки в main/master/..., не мержим его и не закрываем, а в комментариях указываем @PanovaElena В общем, ничего сложного, нужно только быть внимательным. По коду: все нормально, только обычно функция получения указателя в итераторе называется просто get(), можно назвать getPtr() или как-нибудь еще, но getit() сбивает с толку. Принято. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub , or unsubscribe .