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

Add smithy-docgen to the repository #2488

Merged
merged 8 commits into from
Dec 18, 2024
Merged

Conversation

haydenbaker
Copy link
Contributor

Background

  • Adds smithy-docegen to the repository
  • Previously, this was not easily possible with how the repository was structured and built
  • The core module was renamed from smithy-docgen-core to smithy-docgen to fit our conventional naming scheme
    • The actual module structure was not changed, everything is still under core (up for debate whether to change)
  • Reviewers: Please double-check that surface area is as small as possible (everything that can be package-protected or marked as unstable is)

Testing

  • Migrated the -tests package to a couple of simple integration (sanity-check) tests
    • The actual example usage (as a project) should be moved to smithy-examples

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@haydenbaker haydenbaker requested a review from a team as a code owner December 16, 2024 20:00
smithy-docgen/build.gradle.kts Show resolved Hide resolved
smithy-docgen/build.gradle.kts Outdated Show resolved Hide resolved
smithy-docgen/README.md Outdated Show resolved Hide resolved
smithy-docgen/README.md Outdated Show resolved Hide resolved
@haydenbaker haydenbaker merged commit 89f26d6 into main Dec 18, 2024
8 checks passed
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.

3 participants