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 feature gates and serde related types #2651

Closed

Conversation

thomas-k-cameron
Copy link
Contributor

@thomas-k-cameron thomas-k-cameron commented Apr 27, 2023

Motivation and Context

Adds RFC30 related data types and functions to kotiln code.

This PR does NOT introduce changes to code gen logic.

This is a child PR of

The changes that this PR introduces is same as the ones that were merged to unstable-serde-support branch before.

Initially, we tried to make commit to unstable-serde-support branch and merge changes one by one in small PRs. However, in order to make it up to date with the main branch, we would need to go through a large PR of over 700 files.

Thus, I decided to create individual PRs that commits directly to main branch.

Testing

NA

Checklist

NA


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@thomas-k-cameron thomas-k-cameron mentioned this pull request Apr 27, 2023
2 tasks
@thomas-k-cameron thomas-k-cameron changed the title RFC30 : Add feature gates related types to RustType.kt Add feature gates and serde related types to RustType.kt Apr 27, 2023
@thomas-k-cameron thomas-k-cameron changed the title Add feature gates and serde related types to RustType.kt Add feature gates and serde related types Apr 27, 2023
jdisanti added a commit that referenced this pull request Jun 14, 2023
## Motivation and Context
This is a child PR of #2615.

## Description
- Adds `send_with` method to Fluent Builder.

## Prerequisite PRs
You can merge this first too reduce diffs.

- #2651

## Testing
NA

## Checklist
NA

----

_By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice._

---------

Co-authored-by: John DiSanti <johndisanti@gmail.com>
Co-authored-by: John DiSanti <jdisanti@amazon.com>
@thomas-k-cameron
Copy link
Contributor Author

thomas-k-cameron commented Jun 20, 2023

Change this PR was going to introduce was merged else-where.

Closing.

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.

1 participant