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

Fix javaJaxRS template: remove deprecated in JDK9 method usage #18728

Merged

Conversation

rozdy
Copy link
Contributor

@rozdy rozdy commented May 21, 2024

Small fix for java JaxRS template. newInstance() method is deprecated in Class class since JDK9.

@wing328
Copy link
Member

wing328 commented May 22, 2024

can you please follow step 3 to update the samples so that CI can test the change?

@rozdy
Copy link
Contributor Author

rozdy commented May 22, 2024

can you please follow step 3 to update the samples so that CI can test the change?

Done.

@wing328
Copy link
Member

wing328 commented May 26, 2024

tested locally and confirmed the using deprecated api warning is gone. thanks for the fix.

@wing328
Copy link
Member

wing328 commented May 26, 2024

cc
@bbdouglas (2017/07) @sreeshas (2017/08) @jfiala (2017/08) @lukoyanov (2017/09) @cbornet (2017/09) @jeff9finger (2018/01) @karismann (2019/03) @Zomzog (2019/04) @lwlee2608 (2019/10) @martin-mfg (2023/08)

@wing328 wing328 merged commit 0daf9ff into OpenAPITools:master May 26, 2024
36 checks passed
@wing328 wing328 added Server: Java Enhancement: Code Cleanup General refactoring, removal of deprecated things, commenting, etc. labels May 26, 2024
@wing328 wing328 added this to the 7.7.0 milestone May 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement: Code Cleanup General refactoring, removal of deprecated things, commenting, etc. Server: Java
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants