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

Enforce right method parameters order in Javadoc #847

Closed
HDouss opened this issue May 26, 2017 · 25 comments
Closed

Enforce right method parameters order in Javadoc #847

HDouss opened this issue May 26, 2017 · 25 comments

Comments

@HDouss
Copy link
Contributor

HDouss commented May 26, 2017

For now, qulice accepts such javadoc -notice arg related javadoc precedes param related javadoc, although param parameter is declared before arg parameter in the signature-:

    /**
     * Constructor.
     * @param arg Arg
     * @param param Param
     */
    public MyClass(final String param, final String arg) {
        this.param = param;
        this.arg = arg;
    }

It would be better to enforce developers to write method (or constructors) parameterss javadoc in the order they appear in the signature.
Many times, the javadoc misleads about the order for which to call the method, especially when the parameters are in the same type.

@0crat
Copy link
Collaborator

0crat commented May 26, 2017

@krzyk please, pay attention to this issue

@HDouss
Copy link
Contributor Author

HDouss commented Jul 31, 2017

@krzyk Any news here ?

@krzyk
Copy link
Collaborator

krzyk commented Aug 3, 2017

@HDouss currently project is not founded, so no news

@krzyk
Copy link
Collaborator

krzyk commented Apr 29, 2018

@0crat in

@0crat 0crat added the scope label Apr 29, 2018
@0crat
Copy link
Collaborator

0crat commented Apr 29, 2018

@0crat in (here)

@krzyk Job #847 is now in scope, role is DEV

@0crat
Copy link
Collaborator

0crat commented Apr 29, 2018

Bug was reported, see §29: +15 point(s) just awarded to @HDouss/z

@0crat
Copy link
Collaborator

0crat commented Apr 29, 2018

@krzyk/z everybody who has role DEV are banned at this job; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@0crat
Copy link
Collaborator

0crat commented May 4, 2018

@krzyk/z everybody who has role DEV are banned at this job; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@0crat
Copy link
Collaborator

0crat commented May 10, 2018

@krzyk/z everybody who has role DEV are banned at this job; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@0crat
Copy link
Collaborator

0crat commented May 15, 2018

@krzyk/z everybody who has role DEV are banned at #847; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@0crat
Copy link
Collaborator

0crat commented May 20, 2018

@krzyk/z everybody who has role DEV are banned at #847; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@0crat
Copy link
Collaborator

0crat commented May 25, 2018

@krzyk/z everybody who has role DEV is banned at #847; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@0crat
Copy link
Collaborator

0crat commented May 30, 2018

@krzyk/z everybody who has role DEV is banned at #847; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@0crat
Copy link
Collaborator

0crat commented Jun 4, 2018

@krzyk/z everybody who has role DEV is banned at #847; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@0crat
Copy link
Collaborator

0crat commented Jun 9, 2018

@krzyk/z everybody who has role DEV is banned at #847; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@0pdd
Copy link
Collaborator

0pdd commented Jan 23, 2019

@HDouss the puzzle #1001 is still not solved.

@paulodamaso
Copy link
Contributor

@HDouss Merged in #986, please close.

@HDouss
Copy link
Contributor Author

HDouss commented Jan 24, 2019

@paulodamaso Thanks.

@HDouss HDouss closed this as completed Jan 24, 2019
@0crat
Copy link
Collaborator

0crat commented Jan 24, 2019

@ypshenychka/z please review this job completed by @paulodamaso/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

@0crat 0crat removed the scope label Jan 24, 2019
@0crat
Copy link
Collaborator

0crat commented Jan 24, 2019

The job #847 is now out of scope

@ypshenychka
Copy link

@0crat quality good

@0crat
Copy link
Collaborator

0crat commented Jan 25, 2019

@0crat quality good (here)

@ypshenychka The project doesn't have enough funds, can't make a payment

@0crat
Copy link
Collaborator

0crat commented Jan 25, 2019

Order was finished, quality is "good": +35 point(s) just awarded to @paulodamaso/z

@0crat
Copy link
Collaborator

0crat commented Jan 25, 2019

Quality review completed: +8 point(s) just awarded to @ypshenychka/z

@0pdd
Copy link
Collaborator

0pdd commented Jan 30, 2019

@HDouss the only puzzle #1001 is solved here.

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

No branches or pull requests

6 participants