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

Make API simpler and smaller #78

Merged
merged 4 commits into from
Dec 22, 2023

Conversation

tertsdiepraam
Copy link
Member

@tertsdiepraam tertsdiepraam commented Dec 19, 2023

  • We always want to use try_parse so remove parse and rename try_parse to parse. This makes the tests more verbose with unwrap but this is the behaviour we are gonna need in uutils, by making this implement UError and then we use ?.
  • I've removed ArgumentIter from the public API, because it was only cluttering the docs and with some small changes did not need to be public.
  • help now returns a string instead of writing to stdout.
  • Make Value derive not fail without #[value] attributes

Closes #77
Closes #74

@tertsdiepraam tertsdiepraam merged commit fc29ec5 into uutils:main Dec 22, 2023
4 checks passed
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.

try_parse should be the default Value derive macro without value attributes should have a better error
1 participant