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.

Sign up for GitHub

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

Jump to bottom

Update ref docs regarding deprecated @Required annotation #26578

Closed
wants to merge 1 commit into from
Closed

Update ref docs regarding deprecated @Required annotation #26578

wants to merge 1 commit into from

Conversation

alex-krav
Copy link
Contributor

Please select one of the following:

  • add Note to RequiredAnnotationBeanPostProcessor
  • since Required annotation is deprecated, use AutowiredAnnotationBeanPostProcessor to illustrate the example

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Feb 20, 2021
@sbrannen sbrannen changed the title Updated ref docs for Required annotation Update ref docs regarding deprecated @Required annotation Feb 22, 2021
@sbrannen sbrannen self-assigned this Feb 22, 2021
@sbrannen sbrannen added in: core Issues in core modules (aop, beans, core, context, expression) type: documentation A documentation task and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Feb 22, 2021
@sbrannen sbrannen added this to the 5.3.5 milestone Feb 22, 2021
@sbrannen sbrannen closed this in 90ee22f Feb 22, 2021
sbrannen added a commit to sbrannen/spring-framework that referenced this pull request Feb 22, 2021
@sbrannen
Reduce focus on RequiredAnnotationBeanPostProcessor in ref docs
6d2fec3 
See spring-projectsgh-26578
@sbrannen
Copy link
Member

This has been merged into master in 90ee22f and revised in 6d2fec3.

Thanks

@alex-krav
Copy link
Contributor Author

alex-krav commented Feb 23, 2021 •
edited

@sbrannen, I meant that we need one BPP example: RequiredAnnotationBeanPostProcessor with deprecation note or AutowiredAnnotationBeanPostProcessor. You merged both. Maybe that's okay too.

@sbrannen
Copy link
Member

sbrannen commented Feb 23, 2021 •
edited

I meant that we need one BPP example: RequiredAnnotationBeanPostProcessor with deprecation note or AutowiredAnnotationBeanPostProcessor.

Since I updated the cross reference in Annotation-based Container Configuration, I agree that the "Example: The RequiredAnnotationBeanPostProcessor" section is no longer necessary, and I will remove it.

Cheers

See f7cef2a for the latest polishing commit.

sbrannen added a commit to sbrannen/spring-framework that referenced this pull request Feb 23, 2021
@sbrannen
Remove section on RequiredAnnotationBeanPostProcessor in ref docs
f7cef2a 
In order to reduce the focus on the deprecated @required support, this
commit removes the RequiredAnnotationBeanPostProcessor section and
replaces it with a TIP and an updated NOTE.

See spring-projectsgh-26578
This was referenced Mar 16, 2021
Closed
Closed
Closed
Merged
Closed
Closed
Open
Closed
Open
Open
Closed
Merged
Closed
Merged
Merged
Merged
This was referenced Mar 17, 2021
Merged
Open
Closed
Merged
Closed
Closed
Closed
Merged
Closed
Closed
Merged
Merged
Merged
Merged
Open
Open
Open
Closed
Open
Open
Open
Closed
Closed
Closed
Zoran0104 pushed a commit to Zoran0104/spring-framework that referenced this pull request Aug 20, 2021
@alex-krav @Zoran0104
Update reference manual regarding deprecated @required annotation
881cd7a 
Closes spring-projectsgh-26578
Zoran0104 pushed a commit to Zoran0104/spring-framework that referenced this pull request Aug 20, 2021
@sbrannen @Zoran0104
Reduce focus on RequiredAnnotationBeanPostProcessor in ref docs
96c5ef9 
See spring-projectsgh-26578
Zoran0104 pushed a commit to Zoran0104/spring-framework that referenced this pull request Aug 20, 2021
@sbrannen @Zoran0104
Remove section on RequiredAnnotationBeanPostProcessor in ref docs
767b116 
In order to reduce the focus on the deprecated @required support, this
commit removes the RequiredAnnotationBeanPostProcessor section and
replaces it with a TIP and an updated NOTE.

See spring-projectsgh-26578
lxbzmy pushed a commit to lxbzmy/spring-framework that referenced this pull request Mar 26, 2022
@alex-krav @lxbzmy
Update reference manual regarding deprecated @required annotation
fcd81c7 
Closes spring-projectsgh-26578
lxbzmy pushed a commit to lxbzmy/spring-framework that referenced this pull request Mar 26, 2022
@sbrannen @lxbzmy
Reduce focus on RequiredAnnotationBeanPostProcessor in ref docs
cb0e0bd 
See spring-projectsgh-26578
lxbzmy pushed a commit to lxbzmy/spring-framework that referenced this pull request Mar 26, 2022
@sbrannen @lxbzmy
Remove section on RequiredAnnotationBeanPostProcessor in ref docs
203f45b 
In order to reduce the focus on the deprecated @required support, this
commit removes the RequiredAnnotationBeanPostProcessor section and
replaces it with a TIP and an updated NOTE.

See spring-projectsgh-26578
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@sbrannen sbrannen

Labels
in: core Issues in core modules (aop, beans, core, context, expression) type: documentation A documentation task
Projects
None yet
Milestone
5.3.5
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@alex-krav @sbrannen @spring-projects-issues