-
Notifications
You must be signed in to change notification settings - Fork 115
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
Sonarqube Fixes #1603 #1604
Sonarqube Fixes #1603 #1604
Conversation
@lukasj Complaining on copyright checks, but I'm not sure why. I updated the year in the modified files. If you could provide a hint, I would be delighted to make the appropriate updates. |
jaxb-ri/core/src/main/java/org/glassfish/jaxb/core/unmarshaller/DOMScanner.java
Outdated
Show resolved
Hide resolved
rebase on current master and it should be good |
eclipse-ee4j#1603 xsom/src/main/java/com/sun/xml/xsom/XSSchema.java Add 'since' and/or 'forRemoval' arguments to the @deprecated annotation. [squid:MissingDeprecatedCheck] jxc/src/main/java/com/sun/tools/jxc/ConfigReader.java Replace the type specification in this constructor call with the diamond operator ("<>"). [squid:S2293] runtime/impl/src/main/java/org/glassfish/jaxb/runtime/v2/runtime/reflect/Lister.java Replace the type specification in this constructor call with the diamond operator ("<>"). [squid:S2293] xsom/src/main/java/com/sun/xml/xsom/impl/scd/SimpleCharStream.java Add 'since' and/or 'forRemoval' arguments to the @deprecated annotation. [squid:MissingDeprecatedCheck] core/src/main/java/org/glassfish/jaxb/core/marshaller/Messages.java Add 'since' and/or 'forRemoval' arguments to the @deprecated annotation. [squid:MissingDeprecatedCheck] xsom/src/main/java/com/sun/xml/xsom/impl/parser/SAXParserFactoryAdaptor.java Add 'since' and/or 'forRemoval' arguments to the @deprecated annotation. [squid:MissingDeprecatedCheck] core/src/main/java/org/glassfish/jaxb/core/unmarshaller/DOMScanner.java Add 'since' and/or 'forRemoval' arguments to the @deprecated annotation. [squid:MissingDeprecatedCheck] codemodel/codemodel/src/main/java/com/sun/codemodel/JAnnotationArrayMember.java Add 'since' and/or 'forRemoval' arguments to the @deprecated annotation. [squid:MissingDeprecatedCheck] runtime/impl/src/main/java/org/glassfish/jaxb/runtime/v2/runtime/Coordinator.java Replace the type specification in this constructor call with the diamond operator ("<>"). [squid:S2293] Signed-off-by: Sanddust <[email protected]> Also-by: Stephen Davidson <[email protected]>
eclipse-ee4j#1603 Updated copyrights in modified files. Signed-off-by: Stephen Davidson <[email protected]>
eclipse-ee4j#1603 Fixed version in @deprecated annotations. Signed-off-by: Steve Davidson <[email protected]>
Done.
Regards,
Steve
…On 2/17/22 07:24, Lukas Jungmann wrote:
@lukasj <https://github.com/lukasj> Complaining on copyright
checks, but I'm not sure why.
rebase on current master and it should be good
—
Reply to this email directly, view it on GitHub
<#1604 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AALDJC6HSPPE7KLQO2IJCVTU3TZH7ANCNFSM5OJZODUQ>.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
#1603
xsom/src/main/java/com/sun/xml/xsom/XSSchema.java
Add 'since' and/or 'forRemoval' arguments to the @deprecated annotation. [squid:MissingDeprecatedCheck]
jxc/src/main/java/com/sun/tools/jxc/ConfigReader.java
Replace the type specification in this constructor call with the diamond operator ("<>"). [squid:S2293]
runtime/impl/src/main/java/org/glassfish/jaxb/runtime/v2/runtime/reflect/Lister.java
Replace the type specification in this constructor call with the diamond operator ("<>"). [squid:S2293]
xsom/src/main/java/com/sun/xml/xsom/impl/scd/SimpleCharStream.java
Add 'since' and/or 'forRemoval' arguments to the @deprecated annotation. [squid:MissingDeprecatedCheck]
core/src/main/java/org/glassfish/jaxb/core/marshaller/Messages.java
Add 'since' and/or 'forRemoval' arguments to the @deprecated annotation. [squid:MissingDeprecatedCheck]
xsom/src/main/java/com/sun/xml/xsom/impl/parser/SAXParserFactoryAdaptor.java
Add 'since' and/or 'forRemoval' arguments to the @deprecated annotation. [squid:MissingDeprecatedCheck]
core/src/main/java/org/glassfish/jaxb/core/unmarshaller/DOMScanner.java
Add 'since' and/or 'forRemoval' arguments to the @deprecated annotation. [squid:MissingDeprecatedCheck]
codemodel/codemodel/src/main/java/com/sun/codemodel/JAnnotationArrayMember.java
Add 'since' and/or 'forRemoval' arguments to the @deprecated annotation. [squid:MissingDeprecatedCheck]
runtime/impl/src/main/java/org/glassfish/jaxb/runtime/v2/runtime/Coordinator.java
Replace the type specification in this constructor call with the diamond operator ("<>"). [squid:S2293]