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

YouTube sample incorrectly formatted at head. #1253

Closed
elharo opened this issue Feb 10, 2021 · 0 comments · Fixed by #1297
Closed

YouTube sample incorrectly formatted at head. #1253

elharo opened this issue Feb 10, 2021 · 0 comments · Fixed by #1297
Labels
priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. samples Issues that are directly related to samples. type: cleanup An internal cleanup or hygiene concern.

Comments

@elharo
Copy link
Contributor

elharo commented Feb 10, 2021

P1 since it breaks the CI checks

Error:  src/main/java/com/example/json/YouTubeSample.java:[8] (header) RegexpHeader: Line does not match expected header line of '^\W+\*?\W*http://www.apache.org/licenses/LICENSE-2.0$'.
Error:  src/main/java/com/example/json/YouTubeSample.java:[16,1] (whitespace) EmptyLineSeparator: 'package' should be separated from previous statement.
Error:  src/main/java/com/example/json/YouTubeSample.java:[21,1] (imports) CustomImportOrder: Extra separation in import group before 'java.io.IOException'
Error:  src/test/java/com/example/json/YouTubeSampleTest.java:[8] (header) RegexpHeader: Line does not match expected header line of '^\W+\*?\W*http://www.apache.org/licenses/LICENSE-2.0$'.
Error:  src/test/java/com/example/json/YouTubeSampleTest.java:[16,1] (whitespace) EmptyLineSeparator: 'package' should be separated from previous statement.
Error:  src/test/java/com/example/json/YouTubeSampleTest.java:[33,1] (imports) CustomImportOrder: Extra separation in import group before 'java.io.IOException'
Error:  src/test/java/com/example/json/YouTubeSampleTest.java:[33,1] (imports) CustomImportOrder: Wrong lexicographical order for 'java.io.IOException' import. Should be before 'org.junit.Test'.
Error:  src/test/java/com/example/json/YouTubeSampleTest.java:[34,1] (imports) CustomImportOrder: Wrong lexicographical order for 'java.io.InputStream' import. Should be before 'org.junit.Test'.
Error:  src/test/java/com/example/json/YouTubeSampleTest.java:[35,1] (imports) CustomImportOrder: Wrong lexicographical order for 'java.util.Map' import. Should be before 'org.junit.Test'.
Error:  src/test/java/com/example/json/YouTubeSampleTest.java:[37,1] (imports) CustomImportOrder: Import statement for 'org.junit.Assert.assertEquals' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line.
Error:  src/test/java/com/example/json/YouTubeSampleTest.java:[38,1] (imports) CustomImportOrder: Import statement for 'org.junit.Assert.assertNotNull' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line.
Error:  src/test/java/com/example/json/YouTubeSampleTest.java:[44] (sizes) LineLength: Line is longer than 100 characters (found 104).
Error:  Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.1.1:check (default-cli) on project google-http-client-snippets: You have 12 Checkstyle violations. -> [Help 1]
Error:  
Error:  To see the full stack trace of the errors, re-run Maven with the -e switch.
Error:  Re-run Maven using the -X switch to enable full debug logging.
Error:  
Error:  For more information about the errors and possible solutions, please read the following articles:
Error:  [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
Error: Process completed with exit code 1.
@elharo elharo added priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: cleanup An internal cleanup or hygiene concern. labels Feb 10, 2021
@product-auto-label product-auto-label bot added the samples Issues that are directly related to samples. label Feb 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. samples Issues that are directly related to samples. type: cleanup An internal cleanup or hygiene concern.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant