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.

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

Reformatage et mise en qualite #36

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jycr
Copy link

@jycr jycr commented Feb 17, 2022

J'ai commencé à préparer des PR, et quand j'ai voulu les proposer, je me suis rendu compte que de nombreuses modifications concernaient du reformatage de code.
En effet, le formatage actuel du code actuel n'est pas uniforme.

Cette PR vise uniquement à reformater l'ensemble de la base de code de manière uniforme afin de réduire au maximum les modifications dans les futures PR des contributeurs.

J'en ai également profité pour corriger certains problèmes de codage remontés par les analyses SonarCloud.

Pour faciliter la relecture de cette PR, je vous encourage vivement à activer l'option 'Hide whitespace' dans Github.

@jycr jycr force-pushed the reformatage-et-mise-en-qualite branch from 8709785 to bfeba1f Compare February 17, 2022 00:08
@didierfred
Copy link
Collaborator

ok pour moi , il faut juste ne pas modifier le code des librairies externes :

  • tests/lib/jasmine-3.3.0/*
  • tests/script/externalLibs/esprima.js
  • script/externalLibs/esprima.js

@jycr jycr force-pushed the reformatage-et-mise-en-qualite branch from bfeba1f to c2b717a Compare March 6, 2022 23:50
@jycr
Copy link
Author

jycr commented Mar 6, 2022

ok pour moi , il faut juste ne pas modifier le code des librairies externes :

* tests/lib/jasmine-3.3.0/*

* tests/script/externalLibs/esprima.js

* script/externalLibs/esprima.js

-> J'ai supprimé les modifications qui avaient été effectuées sur ces fichiers

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