Skip to content

Exo08 PYN#94

Open
ShowYL wants to merge 2 commits into
guillaumeagile:exo08from
ShowYL:exo08
Open

Exo08 PYN#94
ShowYL wants to merge 2 commits into
guillaumeagile:exo08from
ShowYL:exo08

Conversation

@ShowYL

@ShowYL ShowYL commented Sep 22, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread src/12_RefactoringGolf/hole1/kata.ts Outdated
@@ -1,5 +1,4 @@
/* eslint-disable */

const firstRow = 0;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Constante qui utilise un nombres primitif. Il faut le remplacer par un enum ou une valeur typée pour avoir un typage plus fort.

Comment thread src/12_RefactoringGolf/hole1/kata.ts Outdated
/* eslint-disable */

const firstRow = 0;
const secondRow = 1;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Constante qui utilise un nombres primitif. Il faut le remplacer par un enum ou une valeur typée pour avoir un typage plus fort.

Comment thread src/12_RefactoringGolf/hole1/kata.ts Outdated

const firstRow = 0;
const secondRow = 1;
const thirdRow = 2;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Constante qui utilise un nombres primitif. Il faut le remplacer par un enum ou une valeur typée pour avoir un typage plus fort.

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.

2 participants