-
Notifications
You must be signed in to change notification settings - Fork 11
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
Rename 'AuthenticatedEncryption' class to 'Encryption' #12
Conversation
@spewu Could you please review and provide feedback, or ask a current TrustPilot employee to do so? Thank you in advance! |
@refactorsaurusrex thanks - looks good. I will try to get a hold of someone who can merge and release it |
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.
@refactorsaurusrex Thanks for the PR!
Two small change requests, but otherwise it's good to merge! :)
This change allows the class to be referenced without being redundantly qualified with the containing namespace name.
Although these changes are very minor, they will cause errors for package consumers when they upgrade to the latest version.
Done. Thanks! |
@refactorsaurusrex Currently I don't have access to our nuget.org account. I will let you know when this is sorted. |
Renaming this class allows it to be referenced without either of the following undesirable requirements.
using AuthenticatedEncryption;
statement be placed within a consuming project's namespace. (After the rename, the using statement can be placed outside a namespace declaration, which is a common convention for many projects.)Additionally, I bumped the version number to
2.0.0
because this change is breaking and updated the readme with a corrected example.Closes #11.