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

Sprint4/168 missing documentation #176

Merged
merged 12 commits into from
Mar 14, 2015
Merged

Conversation

KraTuX31
Copy link
Contributor

  • Ajout de documentation là où elle était manquante
  • Modification de l'écriture de code de façon à ce que l'on ne franchise pas la limite des 80 caractères en largeur

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 82.38% when pulling 0784240 on sprint4/168-missingDocumentation into 4dc7ec5 on S4.

@KraTuX31
Copy link
Contributor Author

Prochaines PRs : Interdit de merge si c'est mal commenté!

@aroquemaurel
Copy link
Member

N'est-ce pas, @aroquemaurel !

QMap<int, Models::Project> _projects;
QList<int> _removeInCombo;
bool _locked;
QMap<int, Models::Project> _projects; //!<
Copy link
Contributor

Choose a reason for hiding this comment

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

Malheureusement je crois que t'en as oublié un peu ici :)

@manantsoa
Copy link
Contributor

Bon voilà j'ai relu, à part les quelques commentaires de ci de là que j'ai fait, le boulot est excellent ;)
documentationgood

@KraTuX31
Copy link
Contributor Author

Prise en compte de tes remarques (au passage je trouve que Tsoo tu as été très pertinent).
J'ai refait totalement toute la doc des classes dans gui/widgets/delegates/ que j'avais oublié...

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 82.38% when pulling 1d5ccec on sprint4/168-missingDocumentation into 4dc7ec5 on S4.

@Oxynos
Copy link
Member

Oxynos commented Mar 14, 2015

Je pense que tu peux merger du coup ;)

KraTuX31 added a commit that referenced this pull request Mar 14, 2015
@KraTuX31 KraTuX31 merged commit 2e4d2b2 into S4 Mar 14, 2015
@aroquemaurel aroquemaurel deleted the sprint4/168-missingDocumentation branch November 1, 2017 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants