Skip to content

Commit

Permalink
pewpew -d crash (#99)
Browse files Browse the repository at this point in the history
* Fixed an issue with out directories and stats file params

- new versions of Clap panic with invalid utf8 passed to value_of_os()
  - clap-rs/clap#3344

* Added comments to changes

* Added tests for the command line parser

- Split the parsing code into separate functions that can be tested
- Moved the logger init out of the match functions so we can test the cli parsing
- Added tests around the path checks. Found we weren't parsing the try -d the same and fixed it

* Added additional tests for the cli parsing

* Cleaned up the cli tests

* Updated the cargo deny.toml
- Unicode license is allowed under our current whitelist
- Ignoring the time advisory since chrono should not be impacted. chronotope/chrono#602
  • Loading branch information
tkmcmaster committed Sep 9, 2022
1 parent 0f607e9 commit af9568a
Show file tree
Hide file tree
Showing 3 changed files with 450 additions and 46 deletions.
4 changes: 4 additions & 0 deletions deny.toml
Expand Up @@ -13,6 +13,9 @@ vulnerability = "deny"
unmaintained = "warn"
yanked = "warn"
notice = "warn"
ignore = [
"RUSTSEC-2020-0071",
]

[licenses]
private = { ignore = true }
Expand All @@ -21,6 +24,7 @@ allow = [
"Apache-2.0",
"BSD-3-Clause",
"MIT",
"Unicode-DFS-2016",
"Zlib",
]
copyleft = "deny"
Expand Down

0 comments on commit af9568a

Please sign in to comment.