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

ConstantUsageCheck false positive if constant is only used in annotation #447

Closed
longtimeago opened this issue Jul 26, 2015 · 19 comments
Closed
Assignees
Labels

Comments

@longtimeago
Copy link
Contributor

There is a constant:
private static final int MAX = 25;

and it's used in:

    @Timeable(limit = MAX, unit = TimeUnit.SECONDS)
    public void close() {
    ....
    }

Qulice reports Checkstyle violation that Private constant "MAX" is not used (ConstantUsageCheck)


@davvd
Copy link

davvd commented Jul 30, 2015

@yegor256 dispatch this issue please

@yegor256 yegor256 added the bug label Sep 13, 2015
@davvd
Copy link

davvd commented Sep 14, 2015

@longtimeago thank you for the ticket reported, I topped your account for 15 mins, transaction 64984797

@krzyk
Copy link
Collaborator

krzyk commented Nov 2, 2015

@davvd this is urgent

@davvd
Copy link

davvd commented Nov 3, 2015

@davvd this is urgent

@krzyk right, I added "urgent" label

@davvd davvd added the urgent label Nov 3, 2015
@krzyk
Copy link
Collaborator

krzyk commented Nov 9, 2015

@davvd this is postponed

@davvd
Copy link

davvd commented Nov 13, 2015

@davvd this is postponed

@krzyk OK, I put "postponed" label here

@davvd
Copy link

davvd commented Nov 13, 2015

@davvd this is postponed

@krzyk right, I removed "urgent" tags

@davvd
Copy link

davvd commented Nov 13, 2015

@davvd this is postponed

@krzyk all right, we'll find someone else for this task

@davvd davvd removed the urgent label Nov 13, 2015
@krzyk
Copy link
Collaborator

krzyk commented Nov 13, 2015

@davvd not postponed

@krzyk
Copy link
Collaborator

krzyk commented Nov 13, 2015

@davvd assign @krzyk to this task pls

@krzyk
Copy link
Collaborator

krzyk commented Nov 14, 2015

@davvd please see PR #493

@davvd
Copy link

davvd commented Nov 16, 2015

@davvd not postponed

@krzyk OK, I un-labeled it as "postponed" tag

@davvd
Copy link

davvd commented Nov 16, 2015

@davvd assign @krzyk to this task pls

@krzyk done. @krzyk you are assigned to this task. please proceed...

@davvd davvd removed the postponed label Nov 16, 2015
@davvd
Copy link

davvd commented Nov 16, 2015

@davvd please see PR #493

@krzyk thank you, I will check it

@krzyk
Copy link
Collaborator

krzyk commented Nov 16, 2015

@longtimeago code changes have been merged to master, please close this issue

@longtimeago
Copy link
Contributor Author

@krzyk cool, thanks!

@davvd
Copy link

davvd commented Nov 18, 2015

@longtimeago the code made here contains a puzzle 447-94597ea9/#497, which will be resolved soon

@davvd
Copy link

davvd commented Nov 18, 2015

@krzyk done, I added 45 mins in payment 70054404, 26 hours and 20 mins spent to complete this; the bonus for fast delivery (m=1580); +45 to your rating, your total score is +2708

@0pdd
Copy link
Collaborator

0pdd commented Aug 3, 2022

@longtimeago the puzzle #1126 is still not solved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants