NVE / Designsystem

MIT License
1 stars 0 forks source link

Feature/refactor page structure #192

Closed NVEJoel closed 1 month ago

NVEJoel commented 1 month ago

PR

github-actions[bot] commented 1 month ago

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-192.westeurope.5.azurestaticapps.net

github-actions[bot] commented 1 month ago

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-192.westeurope.5.azurestaticapps.net

github-actions[bot] commented 1 month ago

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-192.westeurope.5.azurestaticapps.net

NVEJoel commented 1 month ago

_Jeg merker at i vue filer lint fungerer ikke. Usikker om du har dine egne innstillingene lokalt, eller har du ikke formatering on save? Reply; Det er rart for i andra applikationer fungerar min formatering on save men ikke i designsystem applikationen. Men vi borde chekka det for vi har massa feil nær man kør npm run lint.

Jeg brukar Husky for som kør npm lint for varje commit, påbørjat en dokumentation om det https://nve-wiki.nve.no/display/UTV/Git+og+Husky . Vi kanskje kan implementera det også for designsystemet?

amish1188 commented 1 month ago

_Jeg merker at i vue filer lint fungerer ikke. Usikker om du har dine egne innstillingene lokalt, eller har du ikke formatering on save? Reply; Det er rart for i andra applikationer fungerar min formatering on save men ikke i designsystem applikationen. Men vi borde chekka det for vi har massa feil nær man kør npm run lint.

Jeg brukar Husky for som kør npm lint for varje commit, påbørjat en dokumentation om det https://nve-wiki.nve.no/display/UTV/Git+og+Husky . Vi kanskje kan implementera det også for designsystemet?

hmm. vi kan kanskje se på det sammen når jeg er på kontoret neste uke? Hvis du vil jeg kan gå gjennom alle filene som du jobba på og formattere de før vi pusher commiten til main. Jeg er veldig glad i husky og syns vi burde bruke den i alle prosjekter. Jeg ser at vi ikke kjører lint script i det hele tatt i ds, noe som må sikkert fikses men det kan vi ta i en separat PR

github-actions[bot] commented 1 month ago

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-192.westeurope.5.azurestaticapps.net

NVEJoel commented 1 month ago

_Jeg merker at i vue filer lint fungerer ikke. Usikker om du har dine egne innstillingene lokalt, eller har du ikke formatering on save? Reply; Det er rart for i andra applikationer fungerar min formatering on save men ikke i designsystem applikationen. Men vi borde chekka det for vi har massa feil nær man kør npm run lint. Jeg brukar Husky for som kør npm lint for varje commit, påbørjat en dokumentation om det https://nve-wiki.nve.no/display/UTV/Git+og+Husky . Vi kanskje kan implementera det også for designsystemet?

hmm. vi kan kanskje se på det sammen når jeg er på kontoret neste uke? Hvis du vil jeg kan gå gjennom alle filene som du jobba på og formattere de før vi pusher commiten til main. Jeg er veldig glad i husky og syns vi burde bruke den i alle prosjekter. Jeg ser at vi ikke kjører lint script i det hele tatt i ds, noe som må sikkert fikses men det kan vi ta i en separat PR

Det blir greit! Du får gjerne formattera nå og merga den efter. Jeg har fixat kommentarerna. Vi kan snakke om Husky og lint nesta uke.

github-actions[bot] commented 1 month ago

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-192.westeurope.5.azurestaticapps.net