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

OAS number - decimal should map to float128 in Go #184

Open
baywet opened this issue Oct 8, 2024 · 0 comments
Open

OAS number - decimal should map to float128 in Go #184

baywet opened this issue Oct 8, 2024 · 0 comments
Labels
bug A broken experience type:breaking-change An issue that will result in dependent client projects failing.

Comments

@baywet
Copy link
Member

baywet commented Oct 8, 2024

See this PR for the mappings across languages MicrosoftDocs/openapi-docs#121

Currently decimal in Go maps to float64, which is too short. It should map to float128 instead. This will be a breaking change and requires:

  • A new method in ParseNode
  • A new method in SerializationWriter
  • Implementations across serialization/parse node
  • A new entry in the Go Conventions Service (kiota)
  • Potential new mappings in Go CodeMethodWriter for the serialization and parse node calls.
@github-project-automation github-project-automation bot moved this to Needs Triage 🔍 in Kiota Oct 8, 2024
@baywet baywet added bug A broken experience type:breaking-change An issue that will result in dependent client projects failing. labels Oct 8, 2024
@baywet baywet moved this from Needs Triage 🔍 to Todo 📃 in Kiota Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A broken experience type:breaking-change An issue that will result in dependent client projects failing.
Projects
Status: New📃
Development

No branches or pull requests

1 participant