-
Notifications
You must be signed in to change notification settings - Fork 68
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
refactor: Move source go files in /sdk
package
#1198
Conversation
Signed-off-by: Ivan Ivanov <ivanivanov.ii726@gmail.com>
Signed-off-by: Ivan Ivanov <ivanivanov.ii726@gmail.com>
Signed-off-by: Ivan Ivanov <ivanivanov.ii726@gmail.com>
Signed-off-by: Ivan Ivanov <ivanivanov.ii726@gmail.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
|
Signed-off-by: Ivan Ivanov <ivanivanov.ii726@gmail.com>
Signed-off-by: Ivan Ivanov <ivanivanov.ii726@gmail.com>
Signed-off-by: Ivan Ivanov <ivanivanov.ii726@gmail.com>
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.
I think that we shouldn't name the folder with the source sdk
as the repo itself is the sdk
.
Signed-off-by: Ivan Ivanov <ivanivanov.ii726@gmail.com>
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.
Description
This PR reorganizes the project structure by introducing an
sdk
directory as the primary location for SDK-related functionality.Changes Made
sdk
directory to serve as the central location for SDK source files.sdk
directory.Why Not Sub-Packages?
Why
/sdk
Instead of/src
?aws-sdk-go
andgo-redis
, avoid using/src
and instead organize their files directly in meaningful directories like sdk, pkg, or at the root level.Fixes #1205