TODO before 1.0 release

[mpizenberg]

Here is a list of things that should be nice to fix before a 1.0 release:

• Re-add the install subcommand (not sure if needed for 1.0, maybe wait for 1.1?).
• Check licensing of everything, including vendored code. For rust code, all deps can either be Apache-2.0, BSD-3-Clause, CC0, ISC, MIT or MPL-2.0 which are all fine if we release under BSD-3-Clause. Licenses of elm vendored code are now provided in full.
• Maybe add some end-to-end tests? not sure but at least one might be good for pull requests in CI. done in Add some end-to-end tests #81
• Complete the json report to make it equivalent to the one of elm-test we can add other reports if needed. Done in 0647683 and Json report elm-test-runner#5
• Complete the junit report to make it compatible with the one of elm-test. Some additional fields might be easy to add and nice to have. Done in b8192ac and Junit report elm-test-runner#6
• Try triggering all the unwrap in the code and see if we can improve behavior and output there. We might want to introduce anyhow. see Improve error messages with anyhow #57
• Redo the diagram with the updated names of things in the code architecture. Done in bb03fda
• Vendor some of the dependencies code in elm-test-runner. Done in Vendor Xml dependency elm-test-runner#7
• See if possible to write an upgrade program to change duplicate descriptions. Or at least see if it's possible to get all duplicate at once -> maybe make a PR in elm-explorations/test. Follow Aggregate multiple duplicate descriptions at the same level elm-explorations/test#163
• Try understand why elm-test adds a description prefix with the module name. Is it a commodity thing? See Question about the behavior related to duplicate descriptions error and automatic description level added with the module name? rtfeldman/node-test-runner#493
• Add a root option to specify where the root elm.json is. Done in Add a --root CLI argument #56
• Check the timeouts for the dependency resolution http calls. Increased from 1s to 10s in f4fa0fa

[mpizenberg]

I'll defer the install subcommand and end-to-end tests for the time being.

[mpizenberg]

Here are some issues discovered by lydell

• elm-test-rs init overwrote my elm.json with 2 spaces indentation instead of 4 (Elm uses 4). -> look at how to do that with an alternative to serde_json. Fixed in bd47112
• It also changed the order (of dependencies). Seems to be a mismatch whether - or / is considered to sort first elm/foo vs elm-community/foo -> See if that's a rust/haskell difference in string ordering for BTreeMap or something else. Fixed in Use a struct instead of String for package ids (fix dependencies ordering) #70
• Another difference compared to node-test-runner: 0 tests (either 0 files, or 0 exposed Test values) is an error in node-test-runner, but not en elm-test-rs. Done in 2c231a0
• elm-test-rs --root /nope gives no error. -> Rework CLI with clap instead of pico_args
• elm-test make is missing in elm-test-rs -> will wait after 1.0.0, like install.

[mpizenberg]

Version 1.0.0 released :) #82