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

@SuppressWarnings(PMD.UnusedPrivateField) #30

Merged
merged 1 commit into from
Dec 21, 2015
Merged

Conversation

simonjenga
Copy link
Contributor

#29 This constant is in use. The @SuppressWarnings annotation is not needed.

@dmarkov
Copy link

dmarkov commented Dec 21, 2015

@simonjenga Many thanks for the PR, let me find a reviewer for it

@dmarkov
Copy link

dmarkov commented Dec 21, 2015

@yegor256 review it please, it is a rather short pull request, no need to do a full code review

@yegor256
Copy link
Member

@rultor merge pls

@rultor
Copy link
Contributor

rultor commented Dec 21, 2015

@rultor merge pls

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor rultor merged commit 778155f into jcabi:master Dec 21, 2015
@rultor
Copy link
Contributor

rultor commented Dec 21, 2015

@rultor merge pls

@yegor256 Done! FYI, the full log is here (took me 5min)

@dmarkov
Copy link

dmarkov commented Dec 22, 2015

@yegor256 16 mins was added to your account, many thanks for your contribution (payment 000-7552663a)! The task took 2 hours and 27 mins.; review comments (c=1) added as a bonus; +16 to your rating, your total score is +7404

@dmarkov
Copy link

dmarkov commented Dec 22, 2015

@rultor deploy

@rultor
Copy link
Contributor

rultor commented Dec 22, 2015

@rultor deploy

@dmarkov OK, I'll try to deploy now. You can check the progress here

@rultor
Copy link
Contributor

rultor commented Dec 22, 2015

@rultor deploy

@dmarkov Done! FYI, the full log is here (took me 5min)

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.

None yet

4 participants