Skip to content

feature/web-ci-pr-lint: add ci action to lint the Next application wh… - #278

Draft
jbuiquan wants to merge 2 commits into
mainfrom
feature/web-ci-pr-lint
Draft

jbuiquan wants to merge 2 commits into
mainfrom
feature/web-ci-pr-lint

Conversation

@jbuiquan

@jbuiquan jbuiquan commented Sep 7, 2025

Copy link
Copy Markdown
Collaborator

Description

Github issue : N/A

Cette PR a pour objectif d'ajouter une action CI pour faire le lint de l'application Next sur une PR ciblant main.
Cela permet de détecter les erreurs en amont. (si le build n'a pas été fait avant la PR).

Pour intégrer une action CI qui build, il aurait fallu ajouter le lien vers la database et donc se servir de l'image docker => on reste sur le lint simple.

Comment tester ?

Faire une PR vers la branche main, constater qu'une action CI se lance et lint le code de l'application Next.

Pour faciliter la validation de ma PR

  • J'ai pris connaissance et respecté les règles de contribution
  • Les pre-commit passent
  • Les test unitaires passent
  • Le code modifié fonctionne en local
  • J'ai demandé une peer-review à un autre bénévole du projet
  • J'ai ajouté / mis à jour de la documentation sur outline

@jbuiquan jbuiquan self-assigned this Sep 7, 2025
@jbuiquan jbuiquan added enhancement New feature or request frontend could have labels Sep 7, 2025
@jbuiquan
jbuiquan requested review from Copilot and machbry September 7, 2025 11:37

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a GitHub Actions CI workflow to lint the Next.js application on pull requests to the main branch. The workflow helps catch linting issues early in the development process before code is merged.

  • Adds automated linting for the Next.js application on pull requests
  • Configures workflow to run only when application code or the workflow file itself changes
  • Sets up Node.js environment with dependency caching for efficient CI runs

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread .github/workflows/ci-app-pr.yaml Outdated
@jbuiquan
jbuiquan force-pushed the feature/web-ci-pr-lint branch 3 times, most recently from 8b1eda4 to 2e9723a Compare September 11, 2025 10:34
jbuiquan and others added 2 commits September 18, 2025 22:39
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@jbuiquan
jbuiquan force-pushed the feature/web-ci-pr-lint branch from 2e9723a to 94fa163 Compare September 18, 2025 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants