Maël Obréjan

Add a Dry-run option

6

A dry-run option would be nice (e.g: `./install --dry-run`). It would allow testing new configs without wrecking your home.

While working on #523 I noticed: - some files are not correctly formatted (i.e: running `cargo fmt --check` fails), - a `.rustfmt.toml` file is present, it was added in #210...

Add `init_timed_custom_env()`

1

Add a missing helper function.

Strongly Typed Errors

4

# Feature request ## Is your feature request related to a problem? Please describe. It is currently impossible to differentiate errors, e.g: when a user signs up there is a...

When `index.html` doesn't contain a `head` closing tag SW injection fails silently. However a valid HTML document doesn't require a `head` element to be present ([it can be implicit](https://html.spec.whatwg.org/multipage/semantics.html#the-head-element)). The...

Missing `types` entry in `package.json`

1

You ship type defs but `package.json` is missing a `types` entry, i.e: ```diff "type": "module", "main": "src/index.js", + "types": "types/index.d.ts", "scripts": { "tsc": "tsc", "eslint": "eslint '**/*.js' --fix" }, ```

Remove `Sync` bounds

2

Currently all route handlers needs to be `Send`, `Sync` and `'static`, the default hyper `Executor` only requires `Send` and `'static` bounds (see [this line](https://github.com/hyperium/hyper/blob/v0.14.10/src/common/exec.rs#L82)). The `Sync` bound is currently needed...