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

Improve documentation regarding plugins inherited by the starter-parent #5057

Open
snicoll opened this issue Jan 31, 2016 · 3 comments
Open
Labels
type: documentation A documentation update

Comments

@snicoll
Copy link
Member

snicoll commented Jan 31, 2016

https://twitter.com/gz_k/status/693822039984095232. In particular start-class should be better documented I think. And we also need to make more clear that the repackage goal is invoked automatically.

@snicoll snicoll added the type: documentation A documentation update label Jan 31, 2016
@snicoll snicoll changed the title Improve documentation regarding repackage goal and starter-parent Improve documentation regarding plugins inherited by the starter-parent Jan 31, 2016
@gzoritchak
Copy link

More than documentation, a warning in the logs would save a lot of time:
«You have defined exec.mainClass for the exec:plugin. It will not be used because spring boot ....».

@gzoritchak
Copy link

You don't consider this to be magic, maybe it's not. But it is modifying a very popular plugin making it not working... Maybe you have good reason for that but there should some warnings.

@snicoll
Copy link
Member Author

snicoll commented Jan 31, 2016

We can't issue a log as you're asking. This is just a default configuration of the Maven build. Nothing forces you to use the parent if you don't want to.

If you want to run your app, you should use mvn spring-boot:run.

@philwebb philwebb added this to the Icebox milestone Jul 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: documentation A documentation update
Projects
None yet
Development

No branches or pull requests

3 participants