Ik heb nog nooit van mijn leven een "Pull Request" ingediend op GitHub. Ik wilde het proberen, dus deze video legt het moment vast waarop ik probeer erachter te komen. Eerlijke waarschuwing, dit is geen beknopte, snelle tutorial. Dit ben ik denk bij mezelf en worstel er doorheen.
Mijn idee was dat ik FitVids.js op cdnjs wilde zetten, zodat mensen het daar doorheen kunnen linken. Hun proces om dat te doen is door hun GitHub-repo te forken, hun instructies voor het toevoegen van een nieuwe bibliotheek te volgen en een pull-aanvraag in te dienen.
Dus we doen het!
Dingen waar ik mee worstelde:
- Ik begreep niet of een pull-verzoek een "Git" -ding of een "GitHub" -ding was. Ik ben er nog steeds niet 100% zeker van, maar het lijkt erop dat het een GitHub-ding is, aangezien we het 100% deden via GitHub.com.
- Ik begreep niet wat precies een "pull request" is - ik dacht dat het meer zou zijn alsof je een specifieke commit naar een andere repo zou verplaatsen. We hebben geleerd dat het eigenlijk de hele opslagplaats is die je samen verplaatst (alle commits die sinds je fork zijn gemaakt).
- Een van hun vereisten was dat het lokaal de "npm-test" doorstaat. Het leek niet te slagen, zelfs niet met een ongewijzigde vork van de repo. Uiteindelijk heb ik het eerste pull-verzoek ook verpest met een onjuiste bestandsnaam. Dit mislukte bij de "Travis build" op GitHub.com, wat ik ook niet begrijp (is het hetzelfde als de npm-test?), Maar na het corrigeren van de bestandsnaam werkte het tweede verzoek.
Als follow-up vroegen de eigenaren van de cdnjs-repo mij de niet-verkleinde versie op te nemen in de repo en een "tag" toe te voegen aan de FitVids-repo met de juiste versies. Dus ik moest nog iets heel nieuws leren, maar kreeg het uiteindelijk voor elkaar.
Het Pull Request