gajus / gajus/flow-runtime

Suggestion: linting and flow error checking

Open
#189 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
797
Forks
49
PR merge metrics
No merged PRs in 30d

Description

This is a:

- [ ] Bug Report
- [ ] Feature Request
- [ ] Question
- [x] Other

Which concerns:

- [x] flow-runtime
- [x] babel-plugin-flow-runtime
- [x] flow-runtime-validators
- [x] flow-runtime-mobx
- [x] flow-config-parser
- [x] The documentation website

### What is the current behaviour?

Currently linting and flow returns a ton of errors. And they are never run on CI.

### What is the expected behaviour?

I expect linting rules and flow to be running on CI.

### Suggestions

Fixing flow errors might be a lot of work, but it would definitely be easier to keep all packages in sync with eslint and flow verision similar to other lerna mono repos.

If you guys would like it I can contribute with the following:
- move eslint to root package (one linting installing)
- fix linting errors
- enforcing linting on CI
- move flow-bin and configuration to root package
- clean up all the flow-typed folders as we only need one

Let me know if you are interested in this.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.