# 117: Låt oss försöka göra en "Pull Request" - CSS-tricks

Anonim

Jag har aldrig i mitt liv skickat in en "Pull Request" på GitHub. Jag ville ge det ett skott, så den här videon fångar ögonblicket när jag försöker lista ut det. Rättvis varning, detta är inte en kortfattad, snabb handledning. Det här är jag som tänker för mig själv och kämpar igenom allt.

Min idé var att jag ville sätta FitVids.js på cdnjs så att människor kan länka upp det där. Deras process för att göra det är genom att gaffla deras GitHub repo, följa deras instruktioner för att lägga till ett nytt bibliotek och skicka in en pull-begäran.

Så vi gör det!

Saker jag kämpade med:

  • Jag förstod inte om en pull-begäran var en "Git" -sak eller en "GitHub" -sak. Jag är fortfarande inte 100% säker men det verkar som om det är en GitHub-sak, som vi gjorde det 100% via GitHub.com.
  • Jag förstod inte vad som exakt komponerar en "pull-begäran" - jag trodde att det skulle vara mer som att du flyttade en specifik kommission till en annan repo. Vi lärde oss att det faktiskt är hela repo som du flyttar tillsammans (alla åtaganden gjorda sedan din gaffel).
  • Ett av deras krav var att det klara "npm-testet" lokalt. Det verkade inte passera det även med en oförändrad gaffel av repo. I slutändan skruvade jag upp den första pull-begäran också med ett felaktigt filnamn. Detta misslyckades med "Travis-byggnaden" på GitHub.com, vilket jag inte heller förstår (är det samma som npm-testet?), Men efter att ha fixat filnamnet fungerade den andra begäran.

Som en uppföljning bad ägarna av cdnjs repo att jag skulle inkludera den icke-minifierade versionen i repo och lägga till en "tagg" till FitVids repo med rätt versioner. Så jag var tvungen att lära mig en annan helt ny sak, men slutligen fick jag det gjort.

Dragförfrågan