Skip to content

Conversation

@quaff
Copy link
Contributor

@quaff quaff commented Nov 8, 2024

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Nov 8, 2024
@snicoll
Copy link
Member

snicoll commented Nov 10, 2024

Thanks for the suggestion but if this needs to be described, it should be in the Spring Boot reference doc.

@snicoll snicoll closed this Nov 10, 2024
@snicoll snicoll added status: declined A suggestion or change that we don't feel we should currently apply and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Nov 10, 2024
quaff added a commit to quaff/spring-framework that referenced this pull request Nov 11, 2024
…gistry" from document

>> it should be in the Spring Boot reference doc.

See spring-projects#33857 (comment)
@snicoll
Copy link
Member

snicoll commented Nov 11, 2024

@bclozel see the above for context. I am reopening to get some more feedback as they are other places within our ref doc that have such references.

@snicoll snicoll reopened this Nov 11, 2024
@snicoll snicoll added status: waiting-for-triage An issue we've not yet triaged or decided on and removed status: declined A suggestion or change that we don't feel we should currently apply labels Nov 11, 2024
@bclozel
Copy link
Member

bclozel commented Nov 22, 2024

While we always work towards consistency, I find this PR very frustrating to deal with. The former sentence came from an actual discussion with a library maintainer in #29904. I don't think dedicating our time to discussing this is time well spent.

@bclozel bclozel closed this Nov 22, 2024
@bclozel bclozel added status: declined A suggestion or change that we don't feel we should currently apply and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Nov 22, 2024
@quaff
Copy link
Contributor Author

quaff commented Nov 22, 2024

One of this PR and #33866 should be merged to keep consistency. @bclozel

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

Labels

status: declined A suggestion or change that we don't feel we should currently apply

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants