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.

Sign up for GitHub

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

Jump to bottom

Opti + typo #108

Merged
merged 4 commits into from
Sep 3, 2024
Merged

Opti + typo #108

merged 4 commits into from
Sep 3, 2024

Conversation

Vilerio
Copy link
Contributor

@Vilerio Vilerio commented Sep 2, 2024

@Vilerio Vilerio requested a review from tryon-dev as a code owner September 2, 2024 16:29
@Gabriel29306
Copy link
Contributor

Pourquoi as-tu enlevé tous les commentaires dans getAverage ?

@Vilerio
Copy link
Contributor Author

Vilerio commented Sep 2, 2024

Pourquoi as-tu enlevé tous les commentaires dans getAverage ?

Les noms de fonctions me paraissaient assez explicites, et leur fonctionnement pas si compliqué que ça

Gabriel29306
Gabriel29306 approved these changes Sep 2, 2024
View reviewed changes
Copy link
Contributor

@Gabriel29306 Gabriel29306 left a comment

Choose a reason for hiding this comment

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

LGTM

Rexxt
Rexxt approved these changes Sep 2, 2024
View reviewed changes
Copy link
Contributor

@Rexxt Rexxt left a comment

Choose a reason for hiding this comment

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

globalement LGTM

src/utils/grades/getAverages.ts Show resolved Hide resolved
src/utils/grades/getAverages.ts Show resolved Hide resolved
@Vexcited Vexcited requested a review from ecnivtwelve September 2, 2024 16:37
@ecnivtwelve
Copy link
Contributor

Pourquoi as-tu enlevé tous les commentaires dans getAverage ?

Les noms de fonctions me paraissaient assez explicites, et leur fonctionnement pas si compliqué que ça

Alors certes, mais je trouve que la méthode de calcul parrait assez contre-intuituve pour devoir être précisée (notamment pour les débiles comme moi qui n'y pipent rien en maths)

@Vilerio
Copy link
Contributor Author

Vilerio commented Sep 3, 2024

Pourquoi as-tu enlevé tous les commentaires dans getAverage ?

Les noms de fonctions me paraissaient assez explicites, et leur fonctionnement pas si compliqué que ça

Alors certes, mais je trouve que la méthode de calcul parrait assez contre-intuituve pour devoir être précisée (notamment pour les débiles comme moi qui n'y pipent rien en maths)

Bruh
Bon bah voilà une version commentée par notre ami GPT, pas le temps de le faire ce matin

oriionn
oriionn approved these changes Sep 3, 2024
View reviewed changes
Copy link
Collaborator

@oriionn oriionn left a comment

Choose a reason for hiding this comment

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

LGTM

@Rexxt
Copy link
Contributor

Rexxt commented Sep 3, 2024

Pourquoi as-tu enlevé tous les commentaires dans getAverage ?

Les noms de fonctions me paraissaient assez explicites, et leur fonctionnement pas si compliqué que ça

Alors certes, mais je trouve que la méthode de calcul parrait assez contre-intuituve pour devoir être précisée (notamment pour les débiles comme moi qui n'y pipent rien en maths)

Bruh Bon bah voilà une version commentée par notre ami GPT, pas le temps de le faire ce matin

sérieux ? de l'IA ? par chez nous ?

@Vilerio
Copy link
Contributor Author

Vilerio commented Sep 3, 2024

Pourquoi as-tu enlevé tous les commentaires dans getAverage ?

Les noms de fonctions me paraissaient assez explicites, et leur fonctionnement pas si compliqué que ça

Alors certes, mais je trouve que la méthode de calcul parrait assez contre-intuituve pour devoir être précisée (notamment pour les débiles comme moi qui n'y pipent rien en maths)

Bruh Bon bah voilà une version commentée par notre ami GPT, pas le temps de le faire ce matin

sérieux ? de l'IA ? par chez nous ?

Pour commenter 15 lignes ça devrait aller, mais je l'aurais fait si j'avais pu ce matin ;)

ecnivtwelve
ecnivtwelve approved these changes Sep 3, 2024
View reviewed changes
Copy link
Contributor

@ecnivtwelve ecnivtwelve left a comment

Choose a reason for hiding this comment

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

LGTM, ça fonctionne chez moi dumoins

@tryon-dev tryon-dev merged commit 04c4b16 into PapillonApp:main Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ecnivtwelve ecnivtwelve ecnivtwelve approved these changes

@oriionn oriionn oriionn approved these changes

@Rexxt Rexxt Rexxt approved these changes

@tryon-dev tryon-dev tryon-dev approved these changes

@Gabriel29306 Gabriel29306 Gabriel29306 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@Vilerio @Gabriel29306 @ecnivtwelve @Rexxt @oriionn @tryon-dev