Please add integer positive type!
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 35/100
Research direction
Start by locating the compiler entry point that validates component property types and the existing integer-type tests. Add the positive-integer property type so valid positive values are accepted and non-positive values are reported as compiler errors. Done means the behavior is covered by tests and documented consistently with the other property types.
Written by the indexing model from the issue text.
Description
If you added :integer_positive type for component properties, it would be just great! Very often, components require just such a property, and writing a check on it every time is unnecessary work. It is much better if the compiler informs the coder about the incorrect value.
- Dominant language
- Elixir
- Stars
- 2.1k
- Forks
- 146
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
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.
More from surface-ui/surface
-
Difficulty 1/5 1-3 hours Newbie friendliness 78/100
surface-ui/surface#634 · 1 reaction ·
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
surface-ui/surface#770 ·
-
Website is downOpen
Difficulty 4/5 3-5 days Newbie friendliness 35/100
surface-ui/surface#768 · 4 comments ·
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
surface-ui/surface#762 · 2 comments · 1 reaction ·
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
surface-ui/surface#756 ·
All issues in surface-ui/surface
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
QuinnWilton/argus#5 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
carverauto/serviceradar#5009 ·
Maintainers usually reply within 1 day
-
help-wanted L: docker L: elm L: github:actions L: helm L: ruby:bundler
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
dependabot/dependabot-core#16425 ·
Maintainers usually reply within 2 days
-
bug javascript
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
Maintainers usually reply within 1 day