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

Source for palaute DB schema diagram and palaute state diagram #559

Merged
merged 7 commits into from
Dec 12, 2023

Conversation

pkalliok
Copy link
Contributor

@pkalliok pkalliok commented Nov 9, 2023

Kuvaus muutoksista

https://jira.eduuni.fi/browse/EH-1531

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ä
  • Yli jääneet kehityskohteet on tiketöity

@pkalliok pkalliok force-pushed the pkalliok-palaute-diagram-source branch from 4c99a7c to a5be977 Compare November 13, 2023 07:22
@pkalliok pkalliok force-pushed the pkalliok-palaute-diagram-source branch from a5be977 to 72a5020 Compare November 13, 2023 08:05
@pkalliok pkalliok force-pushed the pkalliok-palaute-diagram-source branch from f6ba587 to 3d9b49d Compare November 15, 2023 13:42
@pkalliok pkalliok requested review from tomikat and severij December 4, 2023 10:35
@pkalliok
Copy link
Contributor Author

Tää on katselmoitu EH-1531-läpikäyntikokouksessa.

@pkalliok pkalliok merged commit d3f11dc into master Dec 12, 2023
1 check passed
@pkalliok pkalliok deleted the pkalliok-palaute-diagram-source branch December 12, 2023 07:39
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