-
Notifications
You must be signed in to change notification settings - Fork 70
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
Add AES GCM Java Utility #23
base: main
Are you sure you want to change the base?
Add AES GCM Java Utility #23
Conversation
Can you share the below elements:
|
Hi Shreyansh, The SonarQube report was generated and some code smells were discovered. The Final report along with the code changes and the below documents will be updated in the repository by Monday EOD.
Thanks. |
…-docs Implement SonarQube changes Add readme.md Add Javadoc
Can you share/attach the reports here along with a test log of atleast one report for Java and Python Test attach files to PR itself |
Attachments:
Javadoc was added to the utilities/docs folder in the commit itself. It can be viewed from this repo's link if GitHub pages are enabled. |
Please add the aes-gcm-utilities docs section inside the aes-gcm-java-utility as having it at utilities end is confusing |
Done, Please check. |
No description provided.