Skip to content
This repository has been archived by the owner on Apr 24, 2024. It is now read-only.

Add Meeting Notes from Scrum Meeting 04.04.2024 #1245

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion doc/changelog.md
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ Syntax: `- short text describing the change _(Your Name)_`
- Added Meeting Agenda&Notes for 11.03.2024 9:00 _(Markus Raab, Jannis)_
- Added Meeting Agenda&Notes for 25.03.2024 9:00 _(Markus Raab, Moritz)_
- _()_
- _()_
- Added Meeting Notes for 04.04.2024 14:00 _(Andrei Dinu)_
- Add 'Christoph Schreiner' as team member _(Christoph Schreiner)_
- _()_
- Migrate from Jest to Vitest, update Vite to v5, update Node to 20, .env should be .env.local _(Paul)_
Expand Down
43 changes: 43 additions & 0 deletions doc/meetings/2024_04_04_scrum_processes.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
# Meeting 04.04.2024 - Scrum Processes

## Participants

- Markus
- Andrei
- Yvonne

## Questions

The main questions to answer:

- How can we improve the way we do issues?
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- How can we improve the way we do issues?
- How can we improve the way we create and work on issues?

- How can we improve the way we deal with Reviews of PR
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- How can we improve the way we deal with Reviews of PR
- How can we improve the way we deal with reviews of PRs?


Takeaways:

- Every 3 weeks, include a 30min retro in the meeting by Andrei
- After releases, have small Demo so team is up to date with what's new.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- After releases, have small Demo so team is up to date with what's new.
- After releases, we have small demo so team is up to date with what's new.

- Explore how GitLab can help us structure issues around Use Cases/Features
- Try to have only isses that have an estimation of one week (can be done in the scope of one "sprint") - If they are too big, they should be broken down into sub-tasks
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Try to have only isses that have an estimation of one week (can be done in the scope of one "sprint") - If they are too big, they should be broken down into sub-tasks
- Try to have only issues that have an estimation of one week (can be done in the scope of one "sprint") - If they are too big, they should be broken down into sub-tasks

- Reviews should be more structured - what type of review is it? (what should be checked - code/documentation/testing etc.), what issue is it relating to
- Submissions README.md as source of "expertise" - reviewers that match the expertise needed
- Look into having separate "Master" and "Dev" Branches

## Meeting Notes

General points brought up and short discussion on each:

- Timeboxes from Scrum
- Don't fully make sense in the scope of the project, sprints can't be longer than 1 week if there are no other regular touchpoints (e.g Daily Scrum)
- Retro/Review
- Might be interesting to try and integrate those concepts in our current meeting flow
- From Use Cases to Issues to Pull Requests
- We need a better structure, like the concept of "Epic"s or "Feature"s to categorize the issues. This might be better supported by moving to GitLab
- What kind of info should an issue contain? From Use Cases to everything needed for implementation - currently, details might be missing that we only become aware of later
- PRs and Reviews
- Important to have a clear linking between the PR and the task it relates to, clear indication of what the Scope of the PR is, try not to combine multiple issues into one PR
- Who should review? - we can use information from the submissions readme and have some rules of assigning reviews based on expertise - who can most help you review the part of the application you changed
- PRs should not stay open for too long, to keep the Branches more stable and avoid conflicts
- We should clean the older/"orphaned" PRs
- Task Estimation
- In the past, explicit estimation did not work well for the structure of the project. Try to stick to Issue size that is managable in the "sprint" time of one week