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

Create Definition of Ready en Definition of Done #5

Closed
wants to merge 10 commits into from

Conversation

RikHooft007
Copy link
Member

Zoals besproken.

@RikHooft007 RikHooft007 changed the title Create Definition of Ready.md Create Definition of Ready en Definition of Done Dec 2, 2024
@RikHooft007 RikHooft007 self-assigned this Dec 4, 2024
@berrydenhartog berrydenhartog force-pushed the Definition-of-Ready-branch branch from 37725e7 to ce801e5 Compare December 4, 2024 08:55
@RikHooft007 RikHooft007 force-pushed the Definition-of-Ready-branch branch from ce801e5 to 53f99c0 Compare December 4, 2024 12:48
@RikHooft007 RikHooft007 requested a review from a team as a code owner December 4, 2024 12:48
Copy link
Member

@bvhme bvhme left a comment

Choose a reason for hiding this comment

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

Tof Rik,

Hier onder wat opmerkingen. Ook is het denk ik goed om de bestanden in de openbsw map te zetten en om de bestandsnamen daar op aan te passen. Ik stel voor openbsw/definition-of-ready.md en openbsw/definition-of-done.md.

@berrydenhartog, heb jij nog wat in de pap te brokkelen?

Comment on lines 3 to 11
1. Issue is getest en werkt

2. Issue is zo gedocumenteerd of gecodeerd zodat deze reproduceerbaar is als we naar een andere omgeving gaan dan Digilab

3. Architectuur is bijgewerkt

3. Issue is gedocumenteerd in relatie tot BSW en aanbevelingen en selectie projecten

4. Issue is gedemonstreerd aan het team en/of belanghebbende
Copy link
Member

Choose a reason for hiding this comment

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

Lijstjes in Markdown behoeven geen witregel tussen de punten. Er staan ook 2 3. in dit lijstje. Als je in Markdown alles een 1. geeft dan word de lijst automatisch geordend. Ik zie hier echter geen noodzaak voor een volgoorde, dus een ongeordende lijst is beter denk ik.

Suggested change
1. Issue is getest en werkt
2. Issue is zo gedocumenteerd of gecodeerd zodat deze reproduceerbaar is als we naar een andere omgeving gaan dan Digilab
3. Architectuur is bijgewerkt
3. Issue is gedocumenteerd in relatie tot BSW en aanbevelingen en selectie projecten
4. Issue is gedemonstreerd aan het team en/of belanghebbende
* Issue is getest en werkt
* Issue is zo gedocumenteerd of gecodeerd zodat deze reproduceerbaar is als we naar een andere omgeving gaan dan Digilab
* Architectuur is bijgewerkt
* Issue is gedocumenteerd in relatie tot BSW en aanbevelingen en selectie projecten
* Issue is gedemonstreerd aan het team en/of belanghebbende

Een andere suggestie is hier niet issue maar bijvoorbeeld oplossing te gebruiken om het generieker en duidelijker te maken. En ik zou de als we naar een andere omgeving gaan dan digilab vervangen met bijvoorbeeld zodat het herbruikbaar is, inspiratie: https://codefor.nl/community-translations-standard/nl/criteria/make-the-codebase-reusable-and-portable.html

Way of Working/Definition of Ready.md Outdated Show resolved Hide resolved
@RikHooft007
Copy link
Member Author

Hi Boris, ik heb Berry gesproken en zijn feedback (grotendeels 😀) verwerkt. Dat verplaatsen naar openbsw lijkt op een delete + upload, klopt dat?

@@ -0,0 +1,11 @@
**Definition of Done:**
Copy link
Member

Choose a reason for hiding this comment

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

Please use # for headers

# Definition of Ready:

- Oplossing is besproken binnen het team
- Omvang en duur van de oplossing kunnen worden geschat
Copy link
Member

Choose a reason for hiding this comment

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

Omdat we kanban doen en dingen voor het eerst hoeft voor mij de oplossing niet altijd geschat te worden.

@@ -0,0 +1,9 @@
# Definition of Ready:

- Oplossing is besproken binnen het team
Copy link
Member

Choose a reason for hiding this comment

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

Het hoeft voor mij niet met het team besproken te zijn, als het maar goed gedocumenteerd is.

@@ -0,0 +1,5 @@
# Definition of Done:

- Oplossing is getest en werkt
Copy link
Member

@berrydenhartog berrydenhartog Dec 5, 2024

Choose a reason for hiding this comment

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

Ik zou dit veranderen in dat alle technische testen succesvol gedraaid zijn. (dat betekend natuurlijk niet dat het werkt, dat kan je nooit 100% garanderen)


- Oplossing is getest en werkt
- Oplossing is zo gedocumenteerd of gecodeerd zodat deze reproduceerbaar/herbruikbaar is voor andere omgevingen
- Oplossing (ervaring) is gedocumenteerd in relatie tot BSW en aanbevelingen en selectie projecten
Copy link
Member

Choose a reason for hiding this comment

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

Een aantal punten die ik zou willen aandragen:

  • Willen we ook dat alles peer reviewed is?
  • Willen we dat er Security en Quality tools gedraaid worden op onze oplossingen voordat ze door mogen?
  • Willen we dat style guides gehandhaaft worden. Zodat het goed leesbaar blijft en iedereen dezelfde style gebruikt? deze styleguides kunnen we per project vast stellen.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ik denk ook hier dat de teamafspraken meer helder moeten maken hoe we testen en controleren of het werkt, verder:

  1. Geen verkeerd idee, maar doen we dat voor elke oplossing?
  2. Als die er voor alle initiatieven zijn en werken zou ik zeggen: ja.
  3. Idem.

Copy link
Member

Choose a reason for hiding this comment

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

  1. ik zou het doen waar logisch is. Maar bij zoveel mogelijk opleverbare Oplossingen
  2. 👍
  3. 👍


- Oplossing is besproken binnen het team
- Omvang en duur van de oplossing kunnen worden geschat
- Een oplossing is op de juiste manier geadministreerd in GitHub
Copy link
Member

Choose a reason for hiding this comment

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

Wat betekend op de juiste manier geadministreerd? kan je dit verder toelichten

Copy link
Member Author

@RikHooft007 RikHooft007 Dec 6, 2024

Choose a reason for hiding this comment

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

Die vraag is geen verrassing 😄 . Ik stel voor dat we dat apart vastleggen. Zo willen we dat de rapportage onderweg worden opgebouwd zodat we de afgesproken rapporten handig kunnen vullen. Dat vormen onderdeel van de teamafspraken en die duidelijk, vindbaar en begrepen zijn.

@RikHooft007
Copy link
Member Author

As we couldnot remove a commit I close this PR and start a new one on the same subjects

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.

3 participants