Skip to content

feat: add ts-lint workflow#184

Merged
Kout95 merged 6 commits intodevfrom
add-ts-lint-workflow
Apr 9, 2025
Merged

feat: add ts-lint workflow#184
Kout95 merged 6 commits intodevfrom
add-ts-lint-workflow

Conversation

@Kout95
Copy link
Contributor

@Kout95 Kout95 commented Apr 9, 2025

No description provided.

@Kout95 Kout95 requested a review from Copilot April 9, 2025 07:41
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Copy link
Member

@ludovicdmt ludovicdmt left a comment

Choose a reason for hiding this comment

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

Si on veut l'ajouter ça serait plutôt dans ./github/workflows/checks.yml je pense que à part ?

@Kout95
Copy link
Contributor Author

Kout95 commented Apr 9, 2025

Je suis pas sur qu'avoir un gros workflow soit une bonne pratique. Je dirais qu'il vaut mieux plusieurs petits workflow plutot qu'un gros qui met plus de temps à s'exécuter. Et donc à faire des retours.

@Kout95
Copy link
Contributor Author

Kout95 commented Apr 9, 2025

Après si vous préférez le mutualiser je peux changer ça

@Kout95 Kout95 changed the base branch from main to dev April 9, 2025 08:38
@ludovicdmt
Copy link
Member

Dans l'idée oui mais en pratique ça me semble compliqué à maintenir si on crée un fichier YAML par linter ?

Cela dit tu soulèves un bon point que checks.yml est peut être un peu gros (mais il est découpé en jobs qui peuvent runner en simultané donc pas si grave). Mais on gagnerait en clarté et maintnabilité avec un lint.yml séparé d'un test.yml

@Kout95
Copy link
Contributor Author

Kout95 commented Apr 9, 2025

Pour moi check.yml doit se limiter à checker ce qui a besoin d'un back et d'un front en même temps.
Un bon compromis me parait être :

  • check-project.yml => les tests e2e
  • check-front.yml => le lint irait dedans et les test units par exemple
  • check-back.yml => les tests

Le but est de simplifier le(s) workflow(s) et d'avoir des retours plus rapide :)

@ludovicdmt
Copy link
Member

#187

@Kout95 Kout95 merged commit 0575494 into dev Apr 9, 2025
8 checks passed
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.

4 participants