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.

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

[LOGBACK-1583] Search for @NoAutoStart annotations in ancestor hierarchy, implemented interfaces and meta-annotations #531

Merged
merged 2 commits into from
Apr 11, 2024

Conversation

brenuart
Copy link
Contributor

Adresses issue LOGBACK-1583

…d interfaces and meta-annotations

Adresses issue LOGBACK-1583

Signed-off-by: Bertrand Renuart <brenuart@gmail.com>
@brenuart brenuart changed the title Search for @NoAutoStart annotations in ancestor hierarchy, implemented interfaces and meta-annotations [LOGBACK-1583] Search for @NoAutoStart annotations in ancestor hierarchy, implemented interfaces and meta-annotations Oct 13, 2021
return (A) ann;
}
}
for (Annotation ann : anns) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While I can see the purpose of looking at super classes. Is there a use for looking at other annotations to search for a given annotation?

@ceki ceki merged commit 4476edd into qos-ch:master Apr 11, 2024
1 of 7 checks passed
@ceki
Copy link
Member

ceki commented Apr 11, 2024

Thank you for this PR.

After merging, I removed support for annotated annotations in commit f32ed30.

@ceki ceki self-assigned this Apr 11, 2024
@ceki ceki added the DONE label Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants