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

Feature/eslint plugin vuejs accessibility #3390

Open
wants to merge 2 commits into
base: v2-bridge
Choose a base branch
from
Open

Feature/eslint plugin vuejs accessibility #3390

wants to merge 2 commits into from

Conversation

VachetVirginie
Copy link
Contributor

@VachetVirginie VachetVirginie commented Feb 9, 2024 •
edited

Description

⚠️ Il est necessaire de supprimer les nodes modules et lancer un pnpm install

Scenario de test:

ajouter la balise vue <img src="toto"> dans un component et lancer pnpm lint

Playground

// Copiez-collez le contenu de votre Playground.vue entier ici

Type de changement

  • Nouvelle fonctionnalité
  • Correction de bug
  • Changement cassant
  • Refactoring
  • Maintenance
  • Documentation
  • Ce changement nécessite une mise à jour de la documentation

Checklist

  • Ma Pull Request pointe vers la bonne branche
  • Mon code suit le style de code du projet
  • J'ai effectué une review de mon propre code
  • J'ai commenté mon code, en particulier dans les parties difficiles à comprendre
  • J'ai apporté les modifications correspondantes à la documentation
  • Mes modifications ne génèrent aucun nouveau warning
  • J'ai ajouté des tests qui prouvent que mon correctif est efficace ou que ma fonctionnalité fonctionne
  • Les tests unitaires passent localement avec mes modifications
  • J'ai mis à jour le fichier Changelog

@VachetVirginie VachetVirginie force-pushed the feature/eslint-plugin-vuejs-accessibility branch 2 times, most recently from d0cbe1e to e3430e9 Compare February 9, 2024 13:34
@DavidFyon DavidFyon linked an issue Feb 9, 2024 that may be closed by this pull request
Ajouter linter a11y + voir ce qui manque par rapport RGAA #3385
Open
@VachetVirginie VachetVirginie force-pushed the feature/eslint-plugin-vuejs-accessibility branch from 278d1f4 to 2570dd7 Compare February 12, 2024 10:22
@VachetVirginie VachetVirginie force-pushed the feature/eslint-plugin-vuejs-accessibility branch from 2570dd7 to a0bf669 Compare February 12, 2024 10:23
@VachetVirginie VachetVirginie changed the title WIP Feature/eslint plugin vuejs accessibility Feature/eslint plugin vuejs accessibility Feb 12, 2024
4dr1en
4dr1en requested changes Feb 14, 2024
View reviewed changes
Copy link
Contributor

@4dr1en 4dr1en left a comment

Choose a reason for hiding this comment

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

Il faudrai corriger les problèmes du linter avec typescript plutôt que modifier la manière dont le projet est typé.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@4dr1en 4dr1en 4dr1en requested changes

@DavidFyon DavidFyon DavidFyon approved these changes

@deraw deraw Awaiting requested review from deraw

@christopheGaon christopheGaon Awaiting requested review from christopheGaon

Requested changes must be addressed to merge this pull request.

Assignees

@VachetVirginie VachetVirginie

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Ajouter linter a11y + voir ce qui manque par rapport RGAA
3 participants
@VachetVirginie @DavidFyon @4dr1en