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

Remove CUE replace statement once openapi changes are merged #91

Open
sdboyer opened this issue Dec 18, 2022 · 1 comment
Open

Remove CUE replace statement once openapi changes are merged #91

sdboyer opened this issue Dec 18, 2022 · 1 comment

Comments

@sdboyer
Copy link
Contributor

sdboyer commented Dec 18, 2022

I had to fork CUE for some fixes - there were a couple key cases that weren't covered (conjuncts in numeric value types, selectors on object types). This substantially impacted Go codegen elsewhere, being the difference between obviously incorrect and probably correct.

Originally posted by @sdboyer in #90 (comment)

@sdboyer
Copy link
Contributor Author

sdboyer commented Dec 18, 2022

cc @IfSentient you'll need to hoist the replace cuelang.org/go statement from thema's go.mod when you update

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

No branches or pull requests

2 participants