Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
SRU2023 v10 #162
base: main
Are you sure you want to change the base?
SRU2023 v10 #162
Changes from all commits
9e6b724
9339793
a56d25c
30574d4
53a5cd2
2353ab0
6c38831
6ef66b0
55fc7dc
c2043d9
7901a64
db6965f
d848bbd
e222be1
06e69d0
eadb987
ce7b218
f27bff1
3e495b7
ca77829
c2dd8b7
f989a6a
1bafbf4
ec42431
bff2ec5
5fb0003
d3c2aa6
e96050d
f38d191
0ca2502
b7cba31
7711f1d
4360f17
ff4d58c
6432a35
ce46ca4
88c1389
450ce7f
4aa5bda
a8a504b
e85e9c7
c083729
7a7f50e
8316563
0d26e06
db7d108
9bd19c9
d3f8865
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
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.
🧹 Nitpick (assertive)
Overall assessment of the changelog
The changelog for Prowide Core demonstrates active development and significant improvements to the library. Key points to note:
FieldEnum
andNarrativeFragment
class enhances the library's functionality.While these changes are generally positive, it's crucial to ensure that the major version updates (Java 11 and Jakarta EE 10) don't introduce compatibility issues for existing users. Thorough testing and possibly a migration guide for users would be beneficial.
Consider the following recommendations:
NarrativeFragment
class and updated field handlers.🧰 Tools
🪛 LanguageTool
🪛 Markdownlint
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.
💡 Codebase verification
Remaining
javax.*
Imports DetectedThe following files still contain imports from
javax.*
packages:src/main/java/com/prowidesoftware/swift/utils/SafeXmlUtils.java
src/main/java/com/prowidesoftware/swift/io/parser/XMLParser.java
Please update these imports to their corresponding
jakarta.*
packages to ensure compatibility and prevent potentialClassNotFoundException
or compilation errors.🔗 Analysis chain
Ensure All Code Imports Are Updated to 'jakarta.*' Packages
The dependencies have been updated from
javax.*
tojakarta.*
packages:jakarta.persistence:jakarta.persistence-api:3.1.0
jakarta.xml.bind:jakarta.xml.bind-api:4.0.0
jakarta.validation:jakarta.validation-api:3.0.2
Please verify that all code imports and usages have been updated accordingly to prevent
ClassNotFoundException
or compilation errors.Run the following script to find any remaining
javax.*
imports in the main source code:🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
Length of output: 1099
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.
💡 Codebase verification
Remaining
javax.*
Imports Detected in Test CodeThe following test files still contain imports from
javax.*
packages:src/test/java/com/prowidesoftware/swift/utils/SafeXmlUtilsTest.java
import javax.xml.XMLConstants;
import javax.xml.parsers.DocumentBuilder;
import javax.xml.stream.XMLInputFactory;
import javax.xml.stream.XMLStreamException;
import javax.xml.stream.XMLStreamReader;
import javax.xml.transform.Transformer;
import javax.xml.transform.TransformerException;
import javax.xml.transform.stream.StreamResult;
import javax.xml.transform.stream.StreamSource;
import javax.xml.validation.Schema;
import javax.xml.validation.SchemaFactory;
src/test/java/com/prowidesoftware/swift/io/parser/XMLParserTest.java
import javax.xml.parsers.DocumentBuilder;
Please update these imports to use
jakarta.*
packages to ensure consistency with the updated test dependencies.🔗 Analysis chain
Ensure Test Code Imports Are Updated to 'jakarta.*' Packages
The test dependencies have also been updated to use
jakarta.*
packages:jakarta.persistence:jakarta.persistence-api:3.1.0
jakarta.validation:jakarta.validation-api:3.1.0
Please verify that all test code imports and usages have been updated to match these dependencies.
Run the following script to find any remaining
javax.*
imports in the test source code:🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
Length of output: 1371