VNG-Realisatie / Haal-Centraal

Repository voor de Haal Centraal GitHub Pages Landingspagina's
https://vng-realisatie.github.io/Haal-Centraal/
Other
0 stars 3 forks source link

genereren faalt op BRP RvIG repo #59

Closed fsamwel closed 3 years ago

fsamwel commented 3 years ago

na het mergen van pull requests faalt generatie-sdks:

 ! [rejected]        master -> master (fetch first)
415
error: failed to push some refs to 'https://github.com/VNG-Realisatie/Haal-Centraal-BRP-Bevragen-RvIG'
416
hint: Updates were rejected because the remote contains work that you do
417
hint: not have locally. This is usually caused by another repository pushing
418
hint: to the same ref. You may want to first integrate the remote changes
419
hint: (e.g., 'git pull ...') before pushing again.
420
hint: See the 'Note about fast-forwards' in 'git push --help' for details.
421
Error: Process completed with exit code 1.
fsamwel commented 3 years ago

@MelvLee kan jij dit oplossen?

MelvLee commented 3 years ago

Het komt doordat 2 actions (generate-sdk's & generate-postman-collection) tegelijk worden getriggered. Als ze klaar zijn doen ze allebei een commit & push, maar een push mag alleen als er geen remote changes zijn. Ik ga de scripts aanpassen.

fsamwel commented 3 years ago

ook op BRP-bevragen heb ik een issue met de lint:

image

zie https://github.com/VNG-Realisatie/Haal-Centraal-BRP-bevragen/pull/806

@MelvLee is dit hetzelfde als waar dit issue over gaat of een nieuw probleem

MelvLee commented 3 years ago

Nee, het is een andere probleem. Daar heb ik al een fix voor gecommit. Alleen wordt die fix niet gebruikt in de P.R.

fsamwel commented 3 years ago

ik heb de pull request nu gemerged, maar krijg dan nog de foutmelding. Ook na handmatig draaien van de oas-lint workflow.

fsamwel commented 3 years ago

kan het zijn dat deze fix in master zit en nog niet in develop?

MelvLee commented 3 years ago

Ik zag het. Ik zie dat de package-lock.json file is aangepast. Die heb ik lokaal opnieuw gegenereerd en nu werkt het weer.