Skip to content

# Code review Rodney #2

@NinaPajonk

Description

@NinaPajonk

Code review Rodney

Name

Nina Pajonk

Class

Tech-1

Feedback

We zijn als team nog druk bezig, alles bij elkaar aan het voegen. En hebben natuurlijk maar 2 features. Het samenwerken gaat goed, dus denk dat alles wel dik in orde komt!

Feedback specifiek op criterium 1 Project Tech

Alle teamleden werken in een eigen branche. We zijn nog niet aan toe gekomen om te mergen. Ook zie ik in je individuele repo dat je gebruikt hebt gemaakt van branches.

Feedback specifiek op criterium 2 Project Tech

Binnen ons team houden we jouw code standaarden aan, ik vind dat ze er netjes uit zien. En heb geen probleem om die aan te houden. De code style standaarden moet alleen nog even gedocumenteerd worden.

Feedback specifiek op criterium 3 Project Tech

Het team documenteert alle calls. En de wiki is netjes en duidelijk ingericht. De repo is geforked, dus de readme van Rodney staat er nog in. In deze read me wordt duidelijk uitgelegd hoe je het kan installeren. Het is alleen nog Nederlands en Engels door elkaar.

Feedback specifiek op criterium 5 Project Tech

Nog niet van toepassing. Nog druk bezig met het team om alles samen te voegen en werkend te krijgen.

Feedback over andere dingen, zoals voor backend of frontend

  • Alleen voor backend moet “session” nog worden toegepast.
  • Engels/ Nederlands nu nog door elkaar.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions