Making the functions in src/commands.rs compatible #26

Open
opened 2023-03-23 05:46:34 +00:00 by emma · 1 comment
Owner

I also briefly considered dropping the output location argument and having the file be written to stdout so that it could written by the shell, [...]

@BlankParenthesis

I really like the idea of having hopper output the initialized hopfile to stdout, and i think that implementation would be superior to the one in the README.

  • Replace Result types with a CommandError type and implement From<CommandError> for (String, u32) so ? can implicitly convert it to the return type of rust_main()
  • Init should output the hopfile contents to stdout

More will be added as necessary.

> I also briefly considered dropping the output location argument and having the file be written to stdout so that it could written by the shell, [...] – @BlankParenthesis I really like the idea of having hopper output the initialized hopfile to stdout, and i think that implementation would be superior to the one in the `README`. - [ ] Replace Result types with a `CommandError` type and implement `From<CommandError> for (String, u32)` so `?` can implicitly convert it to the return type of `rust_main()` - [ ] Init should output the hopfile contents to stdout More will be added as necessary.
emma added this to the Release milestone 2023-03-23 05:46:34 +00:00
emma added the
enhancement
label 2023-03-23 05:46:34 +00:00
BlankParenthesis was assigned by emma 2023-03-23 05:46:34 +00:00
emma added this to the Version 0.1.0 project 2023-03-27 17:01:20 +00:00
Author
Owner

Edited to include new information.

Edited to include new information.
Sign in to join this conversation.
No description provided.