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

feat(type-safe-api): add smithy shape library project for sharing smithy models between apis #875

Merged
merged 1 commit into from
Oct 29, 2024

Conversation

cogwirrel
Copy link
Member

Adds a SmithyShapeLibraryProject which can be used to define Smithy models that can be reused in different APIs.

Previously, one would have to declare a SmithyModelProject to achieve this, but this needed an ignored service to be defined in the model and generated an OpenAPI spec as part of its build, which is unnecessary.

Note this also renames the "Troubleshooting" section of the docs to "FAQ" as it's answering questions rather than explaining common issues.

This change refactors the Smithy projects/components - only snapshot differences for existing smithy projects is the order of files in the .gitignore

Fixes #825

Copy link

nx-cloud bot commented Oct 28, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 3736e5a. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

@cogwirrel cogwirrel force-pushed the feat/smithy-shape-lib branch from 9883988 to 8d57c12 Compare October 28, 2024 01:10
…thy models between apis

Adds a `SmithyShapeLibraryProject` which can be used to define Smithy models that can be reused in
different APIs.

Previously, one would have to declare a `SmithyModelProject` to achieve this, but this needed an
ignored `service` to be defined in the model and generated an OpenAPI spec as part of its build,
which is unnecessary.

Fixes #825
@cogwirrel cogwirrel force-pushed the feat/smithy-shape-lib branch from 8d57c12 to 3736e5a Compare October 29, 2024 00:10
@cogwirrel cogwirrel merged commit 37b4aa2 into mainline Oct 29, 2024
4 checks passed
@cogwirrel cogwirrel deleted the feat/smithy-shape-lib branch October 29, 2024 00:42
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.

[FEATURE] (type-safe-api) common smithy models without defining a service
2 participants