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

sample code for custom-serialization #513

Merged
merged 1 commit into from
Oct 23, 2024
Merged

sample code for custom-serialization #513

merged 1 commit into from
Oct 23, 2024

Conversation

maxday
Copy link
Contributor

@maxday maxday commented Oct 22, 2024

This PR

  • adds code samples for custom-serialization
  • adds a new job in GitHub Action to tests the code samples (build, local invoke, check against 200 status code)

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

@maxday @msailes @andclt
sample code for custom-serialization (#71)
6c45df5 
* example app

* now with annotation serialization

* Add moshi serializer

* Add jackson-jr serializer

* Add fastJson serializer

* Add gson serializer

* Add RequestStreamHandler(GSON) example

* feat: document BYOS

* feat: document BYOS

* feat: document BYOS

* feat: document BYOS

* fix: lint

* fix: bump to java21

* fix: add samples in GitHub actions

* fix: GitHub Actions

* fix: GitHub Actions

* fix: remove gradlew

* fix: bump versions

* fix: test GitHub action failure

* fix: simplify Jackson Sample

* fix: bump dependencies, remove junit, cleanup templates

* fix: remove junit

* fix: use try with resources

---------

Co-authored-by: Mark Sailes <[email protected]>
Co-authored-by: Andrea Culot <[email protected]>
m-rph
m-rph approved these changes Oct 23, 2024
View reviewed changes
Copy link

@m-rph m-rph left a comment

Choose a reason for hiding this comment

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

LGTM

@maxday maxday merged commit dc6e7f6 into main Oct 23, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@m-rph m-rph m-rph approved these changes

@sachin-gokulam sachin-gokulam sachin-gokulam 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
@maxday @sachin-gokulam @m-rph