add quickstart guide to docs #62
Conversation
can be removed since smithy4s-http4s brings this in transitively
| "com.disneystreaming.smithy4s" %% "smithy4s-http4s" % smithy4sVersion.value, | ||
| "com.disneystreaming.smithy4s" %% "smithy4s-http4s-swagger" % smithy4sVersion.value, |
There was a problem hiding this comment.
So technically, smithy4s-http4s-swagger brings in smithy4s-http4s transitively, should we still include both here? My gut says we should leave both.. but wanted to get other thoughts
There was a problem hiding this comment.
Personally I don't care much about explicit dependencies, transitive is fine for me - I'd leave just the one that takes in all the others
There was a problem hiding this comment.
For the sake of this doc, I'd rather keep both.
|
LGTM, what do you think about also showing a client? Maybe a simple main class (IOApp) that hits a single endpoint? |
Yeah that's a good idea, I think it'd be nice to call out that it can do clients right up front like that. |
What I added is based off a simple project I created here: https://github.com/lewisjkl/smithy4s-minimal-example
Happy to move that project into this repo as an example if we want to.