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

error handling #11

Open
5 of 6 tasks
Rudxain opened this issue Jul 12, 2022 · 1 comment
Open
5 of 6 tasks

error handling #11

Rudxain opened this issue Jul 12, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@Rudxain
Copy link
Owner

Rudxain commented Jul 12, 2022

  • Handle possible std{in, out, err} stream errors
  • Avoid panic if file doesn't exist, and print error message to stderr
  • Avoid panic if path is a directory, and eprint
  • Avoid panic if there's no permission to read a file, and eprint
  • Avoid panic if --length <n> cannot be parsed to usize (handled by clap)
  • Return meaningful exit codes
@Rudxain Rudxain added bug Something isn't working enhancement New feature or request labels Jul 12, 2022
@Rudxain Rudxain added the priority IMMEDIATE ATTENTION REQUIRED label Aug 30, 2022
@Rudxain
Copy link
Owner Author

Rudxain commented Aug 30, 2022

I definitely need to read this lmao

@Rudxain Rudxain changed the title Do better error handling Better error handling Sep 2, 2022
Rudxain added a commit that referenced this issue Sep 14, 2022
@Rudxain Rudxain self-assigned this Oct 26, 2022
@Rudxain Rudxain changed the title Better error handling error handling Apr 23, 2024
@Rudxain Rudxain removed enhancement New feature or request priority IMMEDIATE ATTENTION REQUIRED labels Apr 25, 2024
@Rudxain Rudxain removed their assignment Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant