Skip to content

Fetch: Download progress #31

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

Merged
merged 23 commits into from
Oct 20, 2019
Merged

Fetch: Download progress #31

merged 23 commits into from
Oct 20, 2019

Conversation

mitroc
Copy link
Contributor

@mitroc mitroc commented Oct 20, 2019

No description provided.

@jakubdrozdek
Copy link
Contributor

jakubdrozdek commented Oct 20, 2019

@mitroc Wygląda na to, że znów nie zrobiłeś brancha jak należy i masz śmieci w pull requeście.
image

@jakubdrozdek
Copy link
Contributor

Żeby nowo tworzony branch był zsynchronizowany z naszym tutaj masterem, musisz co jakiś czas (najlepiej przed każdym nowym branchem) aktualizować SWOJEGO mastera (na forku). Tutaj masz instrukcję, jak to zrobić za pomocą pull requesta: https://stackoverflow.com/questions/20984802/how-can-i-keep-my-fork-in-sync-without-adding-a-separate-remote/21131381#21131381

@mitroc
Copy link
Contributor Author

mitroc commented Oct 20, 2019

@jakubdrozdek Hmm, no właśnie zrobiłem pull upstream zanim stworzyłem branch do tego tłumaczenia. Ale potem, już w trakcie tłumaczenia pojawiły się na upstreamie nowe rzeczy no i akceptacja poprzedniego PRa, więc znowu zrobiłem pull upstream i domergowałem do tej gałęzi na której pracuję, żeby mieć aktualne zmiany. Może tam jakiegoś sqasha muszę zrobić.
A ten myk z SO też już testowałem i zasadniczo działał, ale coś mi tam nie grało, nie pamiętam już, chyba pokazywał mnie jako tego, który zrobił PR (albo coś w ten deseń).
Przed następnym zrobię sobie jakieś testowe repo na innym koncie i poćwiczę to :).

@jakubdrozdek
Copy link
Contributor

@mitroc W tym przypadku, jako że robisz forka tylko dlatego, że chcesz wrzucać PR-y do tego repo, można zrobić inaczej, dużo prościej. Poza tłumaczeniami nie zależy Ci raczej na zachowaniu jakichkolwiek zmian na swoim forku, więc można go za każdym razem nadpisywać aktualną wersją mastera z tego repo.

Żeby zaktualizować swojego forka, możesz zrobić tak:

  1. git fetch --all
  2. git checkout master
  3. git reset upstream/master --hard
  4. git push -u origin master -f
    Spowoduje to nadpisanie Twojego lokalnego mastera tym z upstreama, nie ważne ile się zmieniło od ostatniej aktualizacji. Po tym możesz stworzyć nowego brancha, który nie będzie miał konfliktów.

Odnośnie sytuacji, gdzie stworzyłeś brancha, ale upstream się zmienił, możesz zrobić coś takiego, będąc na swoim branchu z tłumaczeniem: git rebase upstream/master. Spowoduje to przesunięcie w czasie Twojego brancha, tak jakby powstał on w oparciu o aktualnego mastera, a nie starego.

@jakubdrozdek
Copy link
Contributor

Merguję, ale zanotuj sobie to, co Ci napisałem, żeby kolejny PR był już na czysto 🙂

@jakubdrozdek jakubdrozdek merged commit 0e59e86 into javascript-tutorial:master Oct 20, 2019
@javascript-translate-bot

Thank you 💖 I updated the Progress Issue #1 🎉 🎉 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants