Skip to content
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

Update modules to remove naming inconsistencies and logically group #570

Merged
merged 1 commit into from
Feb 3, 2025

Conversation

hpmellema
Copy link
Contributor

@hpmellema hpmellema commented Jan 30, 2025

Description of changes

Updates repository modules to remove inconsistencies in naming and logically group modules to make project navigation easier.

Summary of changes:

  • Client and server specific modules are nested into a client and server directory, respectively
  • All aws-specific packages are moved to aws directory and names are given a consistent aws- prefix so artifact will have a clear, consistent name
  • RPCv2 Cbor implementations are moved out of aws directory as that protocol is not aws-specific.
  • Some inconsistencies in module naming are resolved
  • Existing codecs are moved into a codecs directory and the RPCv2 cbor codec is renamed cbor-codec as it can be used outside the RPCv2 context

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@hpmellema hpmellema changed the title Update modules to remove naming inconsistencies and logically nest Update modules to remove naming inconsistencies and logically group Jan 30, 2025
@mtdowling
Copy link
Member

Can you summarize what changed? The diff is basically meaningless since it's so big.

@hpmellema hpmellema merged commit 38dcf29 into smithy-lang:main Feb 3, 2025
3 checks passed
@hpmellema hpmellema deleted the nest-modules branch February 3, 2025 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants