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

Honor user-provided enum numbering schemes #114

Merged
merged 4 commits into from
Dec 20, 2024
Merged

Conversation

msprotz
Copy link
Contributor

@msprotz msprotz commented Dec 19, 2024

This should fix #102 and needs the branch of the same name for krml.

I'll wait for circus CI to go back to green before I try this out -- want to make sure I don't break any big use-case.

@msprotz
Copy link
Contributor Author

msprotz commented Dec 20, 2024

Now that circus is back to green (thanks @sonmarcho) I was able to fix a few more things, and now I observe no regressions. I'll merge this right after FStarLang/karamel#507, its companion PR.

@msprotz msprotz enabled auto-merge December 20, 2024 20:08
@msprotz msprotz merged commit 530d979 into main Dec 20, 2024
4 checks passed
@msprotz msprotz deleted the protz_numbered_enums branch December 20, 2024 20:18
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.

Enums are not translated correctly
1 participant