rtfeldman / rtfeldman/node-test-runner
Add elm review for our elm code
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 140
- Forks
- 83
- Avg merge
- 8h 25m
- Merged PRs (30d)
- 5
Description
Edit by @lydell: What’s left of this issue is adding more elm-review rules, such as elm-review-simplify.
I cannot write instructions for this as I have never used elm-review before! First step is to workout the instructions and define our goals here.
It would be great to ensure the highest quality of our elm code.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by learning elm-review and documenting instructions and quality goals for this repository. Evaluate additional rules such as elm-review-simplify against the existing Elm code. Done means the goals are agreed and the selected elm-review rules are added and usable.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- elm
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100