Differences
This shows you the differences between two versions of the page.
|
uso:laboratoare:laborator-08:git-first-pull-request [2023/11/13 17:42] mihai.pacuraru [Crearea unui Pull Request (PR) pe GitHub] fix note format |
uso:laboratoare:laborator-08:git-first-pull-request [2023/11/16 23:24] (current) mihai.pacuraru [Ștergerea branch-ului în urma integrării în branch-ul master] fix .gif path |
||
|---|---|---|---|
| Line 7: | Line 7: | ||
| Spunem că deschidem un Pull Request care urmează să fie integrat într-un anumit branch. De obicei acel branch este **master**. În acest tutorial vom deschide un Pull Request care urmează să fie integrat în branch-ul **master**. | Spunem că deschidem un Pull Request care urmează să fie integrat într-un anumit branch. De obicei acel branch este **master**. În acest tutorial vom deschide un Pull Request care urmează să fie integrat în branch-ul **master**. | ||
| - | În secțiunea ''%%app_dev_branches%%'' am adus modificările de pe branch-ul **add-gitignore** pe branch-ul **master** prin operația **merge**. În această secțiune vom integra schimbările prin intermediul unui **Pull Request**((https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/about-pull-requests | + | În secțiunea ''[[uso/laboratoare/laborator-08#lucrul_pe_branch-uri | Lucrul pe branch-uri]]'' am adus modificările de pe branch-ul **add-gitignore** pe branch-ul **master** prin operația **merge**. În această secțiune vom integra schimbările prin intermediul unui **Pull Request**((https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/about-pull-requests |
| https://www.atlassian.com/git/tutorials/making-a-pull-request | https://www.atlassian.com/git/tutorials/making-a-pull-request | ||
| - | )) urmând pașii din subsecțiunea ''%%app_dev_pr_steps%%''. | + | )) urmând pașii din subsecțiunea ''[[:uso:laboratoare:laborator-08#pasi_pull_request | Pași Pull Request]]''. |
| <note> | <note> | ||
| Line 17: | Line 17: | ||
| - În general, schimbările nu pot fi integrate fără ca PR-ul să fie aprobat. Aprobarea vine în mod normal de la un alt coleg de echipă. | - În general, schimbările nu pot fi integrate fără ca PR-ul să fie aprobat. Aprobarea vine în mod normal de la un alt coleg de echipă. | ||
| - | - Recenzia codului sursă se face ușor pe un Pull Request. Vom vedea în secțiunea ''%%app_dev_pr_reviewers%%'' cum adăugăm recenzenți. | + | - Recenzia codului sursă se face ușor pe un Pull Request. Vom vedea în secțiunea ''[[:uso:laboratoare:laborator-08#recenzii_si_recenzenti| Recenzii și recenzenți]]'' cum adăugăm recenzenți. |
| </note> | </note> | ||
| Line 259: | Line 259: | ||
| După mai multe runde de feedback și aplicarea acestuia, PR-ul va fi gata de integrat în branch-ul **master**. | După mai multe runde de feedback și aplicarea acestuia, PR-ul va fi gata de integrat în branch-ul **master**. | ||
| - | Note | + | <note> |
| Acest scenariu //ping-pong// se va repeta până când voi, împreună cu recenzenții, ajungeți la concluzia că schimbările pot fi integrate. | Acest scenariu //ping-pong// se va repeta până când voi, împreună cu recenzenții, ajungeți la concluzia că schimbările pot fi integrate. | ||
| + | </note> | ||
| Acum vom face operația **merge** a PR-ului. Apăsăm mai întâi butonul ''%%Merge pull request%%'' după care butonul ''%%Confirm merge%%'' ca în imaginea de mai jos. | Acum vom face operația **merge** a PR-ului. Apăsăm mai întâi butonul ''%%Merge pull request%%'' după care butonul ''%%Confirm merge%%'' ca în imaginea de mai jos. | ||
| Line 292: | Line 292: | ||
| Odată făcută operația ''%%merge%%'', putem șterge branch-ul pe care am lucrat apăsând pe butonul ''%%Delete branch%%''. | Odată făcută operația ''%%merge%%'', putem șterge branch-ul pe care am lucrat apăsând pe butonul ''%%Delete branch%%''. | ||
| - | {{./gifs/GitHub-delete-origin-branch.gif}} | + | {{:uso:laboratoare:laborator-04:github-delete-origin-branch.gif?500}} |
| Această operație va șterge branch-ul doar pe GitHub. Ca să ștergem branch-ul **bubble-sort-implementation** și din repository-ul local folosim comanda ''%%git branch -d%%'': | Această operație va șterge branch-ul doar pe GitHub. Ca să ștergem branch-ul **bubble-sort-implementation** și din repository-ul local folosim comanda ''%%git branch -d%%'': | ||
