# 117: Lad os forsøge at lave en "Pull Request" - CSS-tricks

Anonim

Jeg har aldrig i mit liv indsendt en "Pull Request" på GitHub. Jeg ville give det et skud, så denne video fanger øjeblikket, hvor jeg prøver at finde ud af det. Fair advarsel, dette er ikke en kortfattet, hurtig vejledning. Dette er mig, der tænker på mig selv og kæmper igennem det hele.

Min idé var, at jeg ville sætte FitVids.js på cdnjs, så folk kan linke det deroppe. Deres proces til at gøre det er ved at forkaste deres GitHub repo, følge deres instruktioner for at tilføje et nyt bibliotek og indsende en pull-anmodning.

Så vi gør det!

Ting jeg kæmpede med:

  • Jeg forstod ikke, om en pull-anmodning var en "Git"-ting eller en "GitHub"-ting. Jeg er stadig ikke 100% sikker, men det ser ud til, at det er en GitHub-ting, som vi gjorde det 100% gennem GitHub.com.
  • Jeg forstod ikke, hvad der præcist sammensætter en "pull-anmodning" - jeg troede, det ville være mere som om du flyttede en bestemt forpligtelse over til en anden repo. Vi lærte, at det faktisk er hele repoen, som du bevæger dig sammen (alle forpligtelser foretaget siden din gaffel).
  • Et af deres krav var, at det bestod "npm-testen" ting lokalt. Det syntes ikke at passere det selv med en uændret gaffel af repoen. I sidste ende skruede jeg den første pull-anmodning op med et forkert filnavn. Dette mislykkedes "Travis build" på GitHub.com, som jeg heller ikke forstår (er det det samme som npm-testen?), Men efter at have rettet filnavnet fungerede den anden anmodning.

Som opfølgning bad ejerne af cdnjs-repoen om, at jeg medtager den ikke-minificerede version i repoen og tilføjer et "tag" til FitVids-repoen med de korrekte versioner. Så jeg var nødt til at lære en anden helt ny ting, men fik det til sidst gjort.

Pull-anmodningen