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 opt-choice. Other fixes. #27

Merged
merged 17 commits into from
Oct 24, 2023
Merged

Remove opt-choice. Other fixes. #27

merged 17 commits into from
Oct 24, 2023

Conversation

danfuzz
Copy link
Owner

@danfuzz danfuzz commented Oct 24, 2023

This PR removes opt-choice, because (a) all client code has migrated off of it, and (b) its functionality is well-covered by the more general opt-alias.

In addition:

  • Added a handful of missing test cases.
  • Fixed a couple of problems with the general JSON-postprocessing facilities.

@danfuzz danfuzz merged commit 493444a into main Oct 24, 2023
1 check passed
@danfuzz danfuzz deleted the drop-choice branch October 24, 2023 18:58
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