Skip to content

Conversation

@AlexFalappa
Copy link
Contributor

@AlexFalappa AlexFalappa commented Oct 23, 2019

Rename maven spring boot plugin optimizedLaunch property to spring-boot.run.optimizedLaunch to make it more consistent with the other spring-boot:run goal property names.

Rename maven spring boot plugin optimizedLaunch property to spring-boot.run.optimizedLaunch to make it more consistent with the other spring-boot:run goal
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Oct 23, 2019
@snicoll
Copy link
Member

snicoll commented Oct 23, 2019

Good catch @AlexFalappa, thanks! And bummer. I am tempted to fix this in 2.2.1 even if that's potentially a breaking change. Flagging for team attention to see what the rest of the team thinks.

@snicoll snicoll added the for: team-attention An issue we'd like other members of the team to review label Oct 23, 2019
@AlexFalappa
Copy link
Contributor Author

@snicoll yeah I realised that myself, that's a bit unfortunate.

I would be in favour of renaming it now rather than later cause I think not many users will be affected and tooling has yet to catch up. If 2.2.1 is planned shortly no one will notice 😄.

The fact that it is enabled by default helps because turning it off is less frequently needed.

@snicoll snicoll added type: bug A general bug and removed for: team-attention An issue we'd like other members of the team to review status: waiting-for-triage An issue we've not yet triaged labels Oct 23, 2019
@snicoll snicoll self-assigned this Oct 23, 2019
@snicoll snicoll added this to the 2.2.1 milestone Oct 23, 2019
@philwebb philwebb modified the milestones: 2.2.1, 2.2.x Oct 23, 2019
snicoll pushed a commit that referenced this pull request Oct 23, 2019
@snicoll snicoll closed this in 6378863 Oct 23, 2019
@snicoll
Copy link
Member

snicoll commented Oct 23, 2019

We've discussed this and agree with you @AlexFalappa. Thanks again for the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug A general bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants