# 117: Spróbujmy wykonać „pull request” - CSS-Tricks

Anonim

Nigdy w życiu nie przesłałem „Pull Request” na GitHub. Chciałem spróbować, więc ten film przedstawia moment, w którym próbuję to rozgryźć. Uczciwe ostrzeżenie, to nie jest zwięzły, szybki tutorial. To ja myślę sobie i walczę przez to wszystko.

Pomyślałem, że chciałem umieścić FitVids.js na cdnjs, aby ludzie mogli go tam połączyć. Ich proces polega na rozwidleniu repozytorium GitHub, wykonaniu instrukcji dodawania nowej biblioteki i przesłaniu żądania ściągnięcia.

Więc robimy to!

Rzeczy, z którymi się zmagałem:

  • Nie rozumiałem, czy żądanie ściągnięcia było „Git”, czy „GitHub”. Nadal nie jestem w 100% pewien, ale wygląda na to, że jest to sprawa GitHub, ponieważ zrobiliśmy to w 100% przez GitHub.com.
  • Nie rozumiałem, co dokładnie składa się na „żądanie ściągnięcia” - pomyślałem, że byłoby bardziej tak, jakbyś przesunął określone zatwierdzenie do innego repozytorium. Dowiedzieliśmy się, że tak naprawdę jest to całe repozytorium, które przenosicie razem (wszystkie zatwierdzenia dokonane od czasu forka).
  • Jednym z ich wymagań było to, aby przeszedł lokalnie „test npm”. Wydawało się, że nie przechodzi tego nawet przy niezmienionym rozwidleniu repozytorium. Ostatecznie zepsułem pierwsze żądanie ściągnięcia również z niepoprawną nazwą pliku. To nie powiodło się w „kompilacji Travisa” na GitHub.com, której też nie rozumiem (czy to to samo co test npm?), Ale po poprawieniu nazwy pliku drugie żądanie zadziałało.

W ramach kontynuacji właściciele repozytorium cdnjs poprosili mnie o dołączenie niezminifikowanej wersji do repozytorium i dodanie „tagu” do repozytorium FitVids z poprawnymi wersjami. Musiałem więc nauczyć się kolejnej zupełnie nowej rzeczy, ale ostatecznie udało mi się to zrobić.

Żądanie ściągnięcia