Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chore: skru av autodeploy til dev #1127

Merged
merged 3 commits into from
Mar 12, 2024
Merged

Conversation

kristeine
Copy link
Contributor

💰 Hva forsøker du å løse i denne PR'en

  1. Legg til action for å kun bygge ved commits til utviklingsbrancher
  2. Skrur av autodeploy til dev ved alle commits til utviklingsbrancher

🔎️ Er det noe spesielt du ønsker å fremheve?

Er det noe du er bekymret eller usikker på? Beskriv det gjerne her.

✅ Checklist

Har du husket alle punktene i listen?

  • Jeg har testet mine endringer i henhold til akseptansekriteriene/skissene 🕵️
  • Jeg har testet endringene mine i mobilstørrelse, zoom 200%, skalerer riktig med endret tekststørrelse i browser 📱
  • Jeg har skrevet tester. Hvis du ikke har skrevet tester, beskriv hvorfor under 👇
  • Jeg har fikset en bug, og skrevet regresjonstest for denne
  • Jeg har endret søknadskontrakten og modellversjon i Miljø.ts

Jeg har ikke skrevet tester fordi:

🤷‍♀ ️Hvor er det lurt å starte?

F.eks. commit for commit, alt i ett?

💬 Ønsker du en muntlig gjennomgang?

  • Ja
  • Nei

👀 Screen shots

Har det visuelle endret seg? Legg til før- og etterbilder!

@kristeine kristeine force-pushed the chore/skur-av-autodeploy-til-dev branch from 6edc9da to 596e454 Compare March 12, 2024 13:51
@kristeine kristeine marked this pull request as ready for review March 12, 2024 13:58
@@ -33,33 +30,8 @@ jobs:
NODE_AUTH_TOKEN: ${{ secrets.READER_TOKEN }}
run: yarn test

bygg-yarn-for-dependabot:
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gitt at vi kun kjører denne ved workflow_dispatch vil det ikke skje at actor er dependabot 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Og når vi kjører den med workflow_dispatch er det nettopp for å bygge ut til miljø, derfor kjører vi bygg og deploy-jobbene

@kristeine kristeine enabled auto-merge (squash) March 12, 2024 14:12
@kristeine kristeine disabled auto-merge March 12, 2024 14:16
@kristeine kristeine enabled auto-merge (squash) March 12, 2024 14:16
@kristeine kristeine merged commit 1fb1cbc into main Mar 12, 2024
5 checks passed
@kristeine kristeine deleted the chore/skur-av-autodeploy-til-dev branch March 12, 2024 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants