Skip to content

Conversation

restolho
Copy link
Contributor

@restolho restolho commented Nov 8, 2018

#15114

Changes:
Modified DataSourceHealthIndicator details from "hello" to "result" and updated tests accordingly.

Also while doing a clean install with IntelliJ i had several errors about multiple files not being properly formatted on the following location (spring-boot\spring-boot-project\spring-boot-tools\spring-boot-test-support\src\test\java\org\springframework\boot\testsupport). I didn't commit any changes regarding this, since i don't know if this was a local problem or if there is in fact a problem with them.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Nov 8, 2018
@snicoll snicoll added type: enhancement A general enhancement status: on-hold We can't start working on this issue yet and removed status: waiting-for-triage An issue we've not yet triaged labels Nov 9, 2018
@snicoll snicoll added this to the 2.2.x milestone Nov 9, 2018
@snicoll
Copy link
Member

snicoll commented Nov 9, 2018

@restolho thanks for opening your first pull request to Spring Boot. It looks good. We can't merge it just yet as the 2.2.x line is not open. We'll do so once we've created the 2.1.x branch and master has switched to 2.2.0 snapshots.

@snicoll snicoll removed status: on-hold We can't start working on this issue yet labels Nov 17, 2018
@wilkinsona wilkinsona changed the title Change DataSourceHealthIndicator details from "hello" to "result": #15114 Change DataSourceHealthIndicator details from "hello" to "result" Nov 29, 2018
@snicoll snicoll self-assigned this Dec 3, 2018
@snicoll snicoll modified the milestones: 2.2.x, 2.2.0.M1 Dec 3, 2018
@snicoll snicoll closed this in d9586d9 Dec 3, 2018
snicoll added a commit that referenced this pull request Dec 3, 2018
* pr/15136:
  Change DataSourceHealthIndicator details from "hello" to "result"
@snicoll
Copy link
Member

snicoll commented Dec 3, 2018

@restolho thank you very much for making your first contribution to Spring Boot. This is now merged in master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants