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

OY-4608: Muutoksia Lampi-skriptiin #663

Merged
merged 1 commit into from
Jan 2, 2025
Merged

OY-4608: Muutoksia Lampi-skriptiin #663

merged 1 commit into from
Jan 2, 2025

Conversation

tomikat
Copy link
Contributor

@tomikat tomikat commented Jan 1, 2025

Kuvaus muutoksista

Tee assume role ennen jokaista siirtoa, jotta väliaikainen access key ei ehdi vanhentua.

OY-4608

Muistilista PR:n tekijälle ja katselmoijille

Ennen asettamista katselmointiin

  • Build onnistuu ilman virheitä
  • Toiminnallisuuden kattavat yksikkötestit on tehty osana PR:ia
  • PR:n sisältämät muutokset noudattavat sovittuja koodikäytänteitä
  • Koodi on riittävästi dokumentoitu tai se on muuten yksiselitteistä
  • Nimet (muuttujat, funktiot, ...) kuvaavat koodia hyvin

Katselmoijat tarkastavat, että yllä mainitut kohdat toteutuvat

Ennen mergeämistä master-haaralle

  • Vähintään yksi kehittäjä on katselmoinut ja hyväksynyt muutokset
    • Jos muutoksilla voi jotain rikkoessaan olla kauaskantoiset vaikutukset, kannattaa muutokset hyväksyttää useammalla katselmoijalla
  • Katselmoijien esittämät muutosehdotukset on huomioitu
  • Muutokset on testattu QA-ympäristössä
    • Testausohje kirjoitettu
    • Testaus delegoitu OPH:lle mikäli mahdollista
  • Yli jääneet kehityskohteet on tiketöity

@tomikat tomikat merged commit eefe0f7 into master Jan 2, 2025
2 checks passed
@tomikat tomikat deleted the move-assume-role branch January 2, 2025 07:55
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.

1 participant