Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

tswast
Copy link
Contributor

@tswast tswast commented Jun 20, 2016

tswast added 8 commits March 22, 2016 10:25
Fixes checkstyle errors in bookshelf sample.
I modify the checkstyle config to have SAME_PACKAGE imports appear
before third-party imports. I think this better matches the intent of
the import ordering rules in the style guide.
https://google.github.io/styleguide/javaguide.html#s3.3.3-import-ordering-and-spacing
Expecially since it says the `com.google` imports only appear after
static imports if the source file is in the `com.google` package space.
This script will allow us to run tests against a local server for
testing. Since the App Engine and Jetty server running and testing is so
similar, I combined the scripts rather than copying. The only
differences are which plugin to use and what string to wait for before
running curl.

Since the bookshelf sample also needs a `-Plocal` profile argument, I
extended the script to allow passing through arbitrary arguments to
Maven.
Run the local servers and verify they respond with code 200 to the "/"
path.

Note: bookshelf does not yet run in Travis, since we'll need to set up
client secrets credentials.
Also, update the README instructions for how to push changes upstream.
@tswast tswast changed the title Tswast push java repo tools Update java-repo-tools from getting-started-java. Jun 20, 2016
@tswast tswast merged commit 1d046f9 into master Jun 20, 2016
@tswast tswast deleted the tswast-push-java-repo-tools branch June 20, 2016 23:00
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

Successfully merging this pull request may close these issues.

1 participant