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

chore: remove braces for LeftCurlyCheck issue checkstyle #1075

Merged
merged 1 commit into from Jan 18, 2018

Conversation

@vaano94
Copy link
Contributor

@vaano94 vaano94 commented Jan 18, 2018

There is an issue in maven checkstyle project called Issue #5207: add support for LITERAL_CASE and LITERAL_DEFAULT in LeftCurlyCheck which fails to build wercker on pgjdbc project.
I deleted unnecessary braces so the build could be stable.

@vaano94 vaano94 changed the title fix: remove braces for LeftCurlyCheck checkstyle fix: remove braces for LeftCurlyCheck issue checkstyle Jan 18, 2018
@codecov-io
Copy link

@codecov-io codecov-io commented Jan 18, 2018

Codecov Report

Merging #1075 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1075      +/-   ##
============================================
+ Coverage      67.2%   67.21%   +<.01%     
- Complexity     3661     3662       +1     
============================================
  Files           170      170              
  Lines         15621    15621              
  Branches       2522     2522              
============================================
+ Hits          10498    10499       +1     
  Misses         3936     3936              
+ Partials       1187     1186       -1

@vlsi vlsi added this to the 42.2.1 milestone Jan 18, 2018
@vlsi vlsi changed the title fix: remove braces for LeftCurlyCheck issue checkstyle chore: remove braces for LeftCurlyCheck issue checkstyle Jan 18, 2018
@vlsi vlsi merged commit c2664b4 into pgjdbc:master Jan 18, 2018
2 checks passed
davecramer added a commit to davecramer/pgjdbc that referenced this issue Jul 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants