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.

Sign up for GitHub

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

Jump to bottom

0.16.0 merge #460

Merged
merged 26 commits into from
Sep 16, 2022
Merged

0.16.0 merge #460

merged 26 commits into from
Sep 16, 2022

Conversation

daddykotex
Copy link
Contributor

No description provided.

daddykotex and others added 26 commits August 17, 2022 16:33
@daddykotex
Test that client receive set http status code
caabbdb
@daddykotex
Extract code from output if it has @httpResponsecode
158e253
@daddykotex
Remove leftover println
940c1ee
@daddykotex
Merge remote-tracking branch 'origin/main' into dfrancoeur/http-respo…
8b15b19 
…nse-code
@daddykotex
WIP
a568864
@daddykotex
Use ADT to define the schema visitor output
9951cc0
@daddykotex
Avoid asInstanceOf
c57d006
@daddykotex
Looks like a contravariant instance
40ddb6a
@daddykotex
Add has on Hints
b8974b1
@daddykotex
httpResponseCode as a HttpBinding
4c6f05a 
This ensure that the member is excluded from the payload
when serializing
@daddykotex
Refactor to use has instead of get/isDefined
da6abdb
@daddykotex
Merge remote-tracking branch 'origin/main' into dfrancoeur/http-respo…
14c8590 
…nse-code
@daddykotex
Ensure endpoint.code is used as a fallback
03b84b7
@daddykotex
Merge remote-tracking branch 'origin/main' into dfrancoeur/http-respo…
6ceea76 
…nse-code
@daddykotex
Grab status code from response for Client
29909bb
@daddykotex
Merge remote-tracking branch 'origin/main' into dfrancoeur/http-respo…
ea7acfa 
…nse-code

 Conflicts:
	sampleSpecs/pizza.smithy
@daddykotex
Avoid the Primitive.deriving call
bc7b69f
@daddykotex
Change mima target
27c7ccc
@daddykotex
Merge branch '0.16.0' into dfrancoeur/http-response-code
0a47a3f
@daddykotex
Refactor outputToStatus out of the hot path
57072c9
@daddykotex
Merge pull request #417 from disneystreaming/dfrancoeur/mima
7ed8633 
Change mima target
@daddykotex
Make Hints#has final
2def9af
@daddykotex
Move extraction logic out ServerEndpoint
bcdc093
@daddykotex
Merge remote-tracking branch 'origin/0.16.0' into dfrancoeur/http-res…
b7a5e73 
…ponse-code
@Baccata
Merge pull request #377 from disneystreaming/dfrancoeur/http-response…
17d0e13 
…-code

httpResponseCode handling (server side)
@daddykotex
Merge branch '0.16.0' into 0.16.0-merge
0bf909b
lewisjkl
lewisjkl approved these changes Sep 16, 2022
View reviewed changes
Copy link
Contributor

@lewisjkl lewisjkl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@daddykotex daddykotex merged commit 11a764d into main Sep 16, 2022
@Baccata Baccata deleted the 0.16.0-merge branch October 26, 2022 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lewisjkl lewisjkl lewisjkl approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@daddykotex @lewisjkl @Baccata