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

[Python] Ensure we generate correct string literals in Python unions #2478

Merged
merged 5 commits into from
Mar 21, 2023

Conversation

crisidev
Copy link
Contributor

@crisidev crisidev commented Mar 21, 2023

Motivation and Context

We generate type related errors that could not be a valid string literal (they could contain for example [ and ]. This change ensure these error messages are always valid.


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

@crisidev crisidev requested review from a team as code owners March 21, 2023 12:49
@crisidev crisidev requested review from unexge and jjant March 21, 2023 12:49
@crisidev crisidev changed the title [Python] Make sure string literals are valid [Python] Make sure string literals are valid in Union errors Mar 21, 2023
@crisidev crisidev changed the title [Python] Make sure string literals are valid in Union errors [Python] Support more testing models Mar 21, 2023
@crisidev crisidev changed the title [Python] Support more testing models [Python] Ensure we generate correct string literals in Python unions Mar 21, 2023
@github-actions
Copy link

A new generated diff is ready to view.

  • AWS SDK (ignoring whitespace)
  • No codegen difference in the Client Test
  • No codegen difference in the Server Test
  • No codegen difference in the Server Test Python

A new doc preview is ready to view.

@github-actions
Copy link

A new generated diff is ready to view.

  • AWS SDK (ignoring whitespace)
  • No codegen difference in the Client Test
  • No codegen difference in the Server Test
  • No codegen difference in the Server Test Python

A new doc preview is ready to view.

@crisidev crisidev enabled auto-merge March 21, 2023 13:46
@github-actions
Copy link

A new generated diff is ready to view.

  • AWS SDK (ignoring whitespace)
  • No codegen difference in the Client Test
  • No codegen difference in the Server Test
  • No codegen difference in the Server Test Python

A new doc preview is ready to view.

@crisidev crisidev added this pull request to the merge queue Mar 21, 2023
@crisidev crisidev merged commit 2bb969f into main Mar 21, 2023
@crisidev crisidev deleted the oxipy-testing-models branch March 21, 2023 14:34
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.

2 participants