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

Fix package name for mainClassName in build.gradle of spring-boot-sample-web-ui #3588

Closed
wants to merge 1 commit into from
Closed

Conversation

justingarrick
Copy link

The package name for the main class in build.gradle is incorrectly set to:
mainClassName = "sample.ui.SampleWebUiApplication"

It should be set to:
mainClassName = "sample.web.ui.SampleWebUiApplication"

@snicoll
Copy link
Member

snicoll commented Jul 23, 2015

Thanks, have you signed the CLA?

@justingarrick
Copy link
Author

@snicoll I haven't. It says that Before we accept a non-trivial patch or pull request we will need you to sign the contributor’s agreement.

I assume a single line change counts as trivial, no? 😄 Not worried about author credits...

@snicoll
Copy link
Member

snicoll commented Jul 23, 2015

Better be safe than sorry. If that's not a problem, just sign the damn thing please ;)

@justingarrick
Copy link
Author

@snicoll Done!

@snicoll snicoll self-assigned this Jul 23, 2015
@snicoll snicoll added this to the 1.3.0.RC1 milestone Jul 23, 2015
@snicoll snicoll closed this in 8960b88 Jul 23, 2015
snicoll added a commit that referenced this pull request Jul 23, 2015
Since each sample has (supposedly) only one application class and not
a lot of the code, the `mainClassName` attribute in the gradle build is
not really useful and can lead to inconsistency if the application class
is moved.

See gh-3588
snicoll added a commit that referenced this pull request Jul 23, 2015
* pr/3588:
  Remove unnecessary mainClassName
  Fix package name of web-ui sample
@snicoll
Copy link
Member

snicoll commented Jul 23, 2015

Merged, thank you!

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.

None yet

2 participants