GitHub

Alles wat niet past in bovenstaande onderwerpen
Gebruikers-avatar
Berichten: 172
Geregistreerd: 22 Jan 2013, 16:40

GitHub

Berichtdoor zuid » 09 Jan 2020, 07:37

Naar aanleiding van een melding van mij is er een verbetering aangebracht in een ESP8266 bibliotheek.
Om dat het om een hardware combinatie gaat die bij de maker van de verbetering niet direct beschikbaar is, is mij gevraagd het "even" lokaal uit te testen.
Is er hier iemand die ervaring met GitHup heeft die mij een beetje op weg kan helpen?
Er is verwezen naar: https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/checking-out-pull-requests-locally
Mijn scherm geeft niet de informatie die in de instructie wordt omschreven.

Advertisement

Gebruikers-avatar
Berichten: 2520
Geregistreerd: 06 Aug 2016, 01:03

Re: GitHub

Berichtdoor Koepel » 09 Jan 2020, 11:34

Geef ook even een link naar het issue en naar het bericht waarin je gevraagd wordt om het te testen. Welke extra hardware gebruik je ?

Gebruikers-avatar
Berichten: 172
Geregistreerd: 22 Jan 2013, 16:40

Re: GitHub

Berichtdoor zuid » 09 Jan 2020, 13:23

Het gaat om deze vraag: https://github.com/esp8266/Arduino/issues/6992#issuecomment-572056887
(Maar ook om: https://github.com/esp8266/Arduino/issues/6981

zie ook PR #7000 https://github.com/esp8266/Arduino/pull/7000

Het betreft (o.a.) de hardware combinatie een ESP8266 en een SD module en SD FAT16 kaart (2GB).

De instructie schrijft over een knop "command line instructions".
Die knop zie ik niet maar het kan ook zijn dat ik op de verkeerde route zit.

Gebruikers-avatar
Berichten: 2520
Geregistreerd: 06 Aug 2016, 01:03

Re: GitHub

Berichtdoor Koepel » 09 Jan 2020, 18:05

Zo iets heb ik wel eens gedaan, maar nu zie ik het ook niet :?

Die link naar de uitleg van "Checking out pull requests locally" is er voor als je zelf de repository beheerd en iemand anders stuurt een pull request. Er staat duidelijk "... your repository ...", dus dat is per ongeluk de verkeerde link.

Bij commits lijkt er een test-repository te zijn als je rechts op de <> knop klikt. Die zou je kunnen downloaden en testen. Ik begrijp er niet veel meer van, dus controleer even of daar de wijzigingen ook echt in zitten.

Gebruikers-avatar
Berichten: 172
Geregistreerd: 22 Jan 2013, 16:40

Re: GitHub

Berichtdoor zuid » 09 Jan 2020, 18:44

Ik ga daar ook nog eens kijken. Ik denk dat de eigenaar van de source mij rechten moet toekennen anders kunnen er allemaal tussen versies (met vragen) over de gebruikers verspreid raken. Zelf heb ik nu wel de betrokken source met moeite lokaal. (handmatig :) )
Op de pagina met PR#7000 staat een link naar "files changed(9)".
Hier staan de wijzigingen en daar is met enige zoeken ook de gewijzigde source te vinden.
In dit geval 9 bestanden. Waarvan ik er 7 naar mijn PC heb gekopieerd.
Nu nog op de juiste plek plaatsen en kijken of het werkt.

Gebruikers-avatar
Berichten: 172
Geregistreerd: 22 Jan 2013, 16:40

Re: GitHub

Berichtdoor zuid » 09 Jan 2020, 22:23

@koepel, Ter info, het is me gelukt de gewijzigde bibliotheek te testen.

Gebruikers-avatar
Berichten: 172
Geregistreerd: 22 Jan 2013, 16:40

Re: GitHub

Berichtdoor zuid » 10 Jan 2020, 10:30

@Koepel Ik denk dat ik gevonden heb hoe ik het had moeten doen.
Rechts boven aan op een GitHup pagina is een knop FORK.
Dit zegt Google daarvan https://help.github.com/en/github/getting-started-with-github/fork-a-repo

Gebruikers-avatar
Berichten: 2520
Geregistreerd: 06 Aug 2016, 01:03

Re: GitHub

Berichtdoor Koepel » 10 Jan 2020, 11:13

Dat is als je zélf een pull request wilt doen, maar je werd nu gevraagd om de pull request van iemand anders te testen.

Trouwens, als ik eerst een fork maak om een pull request te doen, dan blijft daarna die fork bij mijn repositories staan terwijl ik er weinig meer mee heb. Ik heb na een tijdje zo'n fork maar verwijderd, zonder te weten wat dat met het pull request doet :?

Gebruikers-avatar
Berichten: 172
Geregistreerd: 22 Jan 2013, 16:40

Re: GitHub

Berichtdoor zuid » 10 Jan 2020, 12:14

Is het niet zo dat als ik een fork gedaan zou hebben ik vervolgens de beschikking zou hebben over de software die ik had moeten testen?
Na het testen had ik dan op de originele vraag kunnen reageren en vervolgens de fork weer verwijderen.
Misschien een omweg maar ik heb nog geen betere kunnen vinden.

Gebruikers-avatar
Berichten: 2520
Geregistreerd: 06 Aug 2016, 01:03

Re: GitHub

Berichtdoor Koepel » 10 Jan 2020, 15:20

Als de Pull Request nog niet is uitgevoerd, dan zit dat ook niet in de fork van de originele Master.

Volgens mij kun je bij de Commits de hele repository downloaden waar de Pull Request al wel in is verwerkt. De wijzigingen in de Master van ná de Pull Request zitten daar dan weer niet in.
Bijlagen
github_commits.png
github_commits.png (54.14 KiB) 1762 keer bekeken

Volgende

Terug naar Theekransje

Wie is er online?

Gebruikers in dit forum: Geen geregistreerde gebruikers en 0 gasten