Skip to content

Conversation

nosan
Copy link
Contributor

@nosan nosan commented Aug 9, 2025

See #46688

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Aug 9, 2025
@nosan nosan changed the title Add an ArchRule to ensure that the return type does not have the private modifier Add an ArchRule to ensure that the return type of a @Bean method is not private Aug 9, 2025
@snicoll
Copy link
Member

snicoll commented Aug 10, 2025

@nosan I wonder if that would be possible to restrict this to production code. This would reduce the knock-off effect as we have no intention to invoke AOT processing on our own tests at this time.

@snicoll snicoll added the status: waiting-for-feedback We need additional information before we can continue label Aug 10, 2025
@nosan
Copy link
Contributor Author

nosan commented Aug 10, 2025

@snicoll Agreed, added rules only for main sources.

@spring-projects-issues spring-projects-issues added status: feedback-provided Feedback has been provided and removed status: waiting-for-feedback We need additional information before we can continue labels Aug 10, 2025
@snicoll snicoll self-assigned this Aug 11, 2025
@snicoll snicoll added type: task A general task and removed status: waiting-for-triage An issue we've not yet triaged status: feedback-provided Feedback has been provided labels Aug 11, 2025
@snicoll snicoll added this to the 3.4.9 milestone Aug 11, 2025
snicoll pushed a commit that referenced this pull request Aug 11, 2025
@snicoll snicoll closed this in 95d9b39 Aug 11, 2025
@snicoll
Copy link
Member

snicoll commented Aug 11, 2025

Good stuff @nosan, thanks again!

@nosan
Copy link
Contributor Author

nosan commented Aug 11, 2025

Thanks, @snicoll
May I ask you to close this issue #46688?

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

Labels

type: task A general task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants