Skip to content

Ty/scrum 119 sprint2 iteration#47

Merged
thomasyzy7 merged 24 commits intodevelopfrom
ty/scrum-119-sprint2-iteration
Mar 4, 2025
Merged

Ty/scrum 119 sprint2 iteration#47
thomasyzy7 merged 24 commits intodevelopfrom
ty/scrum-119-sprint2-iteration

Conversation

@thomasyzy7
Copy link
Copy Markdown
Contributor

Description

SCRUM-119

Added iteration-02.plan.md and RPM.md for sprint 2.

Direct google doc link (if you prefer providing more direct feedbacks):

Type of change

  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings

@thomasyzy7 thomasyzy7 changed the base branch from main to develop February 28, 2025 03:21
@kevin-lann kevin-lann self-requested a review February 28, 2025 22:08
Copy link
Copy Markdown
Contributor

@kevin-lann kevin-lann left a comment

Choose a reason for hiding this comment

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

Tiny format fix

Comment thread doc/sprint2/RPM.md Outdated
fixed formatting
@thomasyzy7 thomasyzy7 requested a review from kevin-lann March 2, 2025 19:55
Copy link
Copy Markdown
Contributor

@Austin-X Austin-X left a comment

Choose a reason for hiding this comment

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

Overall, looks good to me. I just left a comment where I found a missing section typo.

Comment thread doc/sprint2/RPM.md
added missing section

Co-authored-by: Austin-X <xaustin03@gmail.com>
@thomasyzy7 thomasyzy7 requested a review from Austin-X March 2, 2025 22:03
Copy link
Copy Markdown
Contributor

@Austin-X Austin-X left a comment

Choose a reason for hiding this comment

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

Looks good to me.

Copy link
Copy Markdown
Contributor

@kevin-lann kevin-lann left a comment

Choose a reason for hiding this comment

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

Looks all good

Copy link
Copy Markdown
Contributor

@MasahisaSekita MasahisaSekita left a comment

Choose a reason for hiding this comment

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

Everything looks good

@thomasyzy7 thomasyzy7 merged commit 69e3275 into develop Mar 4, 2025
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.

5 participants