search

NVE / Designsystem

MIT License
1 stars 0 forks source link

Det bør ikke være lov å pushe til main, men da må vi fikse automatisk bygg og publisering først #112

Open gruble opened 7 months ago

gruble commented 7 months ago

Vi har skrudd av beskyttelse av main, fordi i byggescriptet feiler hvis vi ikke tillater at man pusher til main. Årsaken er at byggescriptet jekker opp versjonsnummer i package.json før den publiserer til npm og deretter pusher den endringa i package.json til main. Vi må finne en annen måte å løse versjonering på.

amish1188 commented 7 months ago

Hmm vi gjør ikke noe med versjonsnummer i byggescriptet. Den er bumpet i Publish package to npmjs action. Er du sikker det er årsaken @gruble ? Jeg trodde at problemet med main var at den var 'restricted' når vi hadde noen regler på den. Hvis det er det som er problemet så kanskje er det ok å ha egen 'publish' eller 'release' branchen som henter endringene fra main og publisere fra den istedenfor.

gruble commented 1 month ago

@amish1188, jeg uttrykte meg litt upresist, Med byggescriptet mente jeg publiseringsscriptet: https://github.com/NVE/Designsystem/blob/ce0ac4a57c56ea68841bb3ef9003c7f179c5d019/.github/workflows/npm-publish.yml#L32 Dette pusher til main idag. Jeg tror ditt forslag til løsning med en egen release-grein vil funke bra. Det er bare å fikse;-)