Daniel Steigerwald

--save-dev

1

In the readme, it should be --save imho.

Lens props should not requires at least two argument

1

I understand it's a helpful API, but in my general form library I have this code: ```ts lens.props(...(Object.keys(action.values) as never)), ``` Which is correct I suppose, but I got `Expected...

Lens set

1

Lens set as modify without a callback. ```ts case 'reinitialize': return pipe( state, pipe( lens.id(), lens.prop('reinitialize'), // lens.set(true), lens.modify(() => true), ), ); ```

Thoughts

14

What do you think about https://github.com/exponentjs/ex-navigation? Isn't it the better approach?Also, I'm not sure whether it makes sense to rely on RR3 when RR4 should be universal and entirely different.

Add flow-type

3

- Nice code, no warnings at all, except one. - Added some types. - Don’t be surprised code is slightly reformatted. I am using awesome jlongster/prettier, everybody should use it....

Idea: Enforce 100% covering by Flow

6

Atom somehow shows how much code is covered by Flow. CleanScript demands 100% I guess.

Web REPL

2

Something like https://babeljs.io/repl, but without any option. The right side should render eslint and flowtype errors. It should help people to write parts of their apps in the clean style....

After #11 of course.

Example lint script not working

2

### What version of Turborepo are you using? None ### What package manager are you using / does the bug impact? pnpm ### What operating system are you using? Mac...