Commands accept arguments they do not use
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 88/100
Research direction
Start in Sources/removemacai/main.swift and compare argument validation in apply with the removemacai tweaks, features, clean, and off commands. Add short error messages and non-zero exits for the three cases described; done when each rejects the invalid arguments consistently with apply.
Written by the indexing model from the issue text.
Description
I noticed a few commands take arguments and then do nothing with them:
removemacai tweaks bogusandremovemacai features extraignore the extra wordremovemacai clean --bogustreats--bogusas a tweak name instead of rejecting it as an option, whichapplydoesremovemacai off --keepwith no value says "unknown option --keep" instead of saying--keepneeds feature names
Each of these should fail with a short message and a non-zero exit, the way apply handles an unknown option. All of it is in Sources/removemacai/main.swift.
- Dominant language
- Swift
- Stars
- 4.9k
- Forks
- 144
- Avg merge
- 19h 52m
- Merged PRs (30d)
- 16
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
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 omlahore/RemoveMacAI
-
enhancement help wanted
Difficulty 3/5 1-2 days Newbie friendliness 67/100
omlahore/RemoveMacAI#61 ·
Maintainers usually reply within 1 day
-
enhancement help wanted
Difficulty 3/5 1-2 days Newbie friendliness 72/100
omlahore/RemoveMacAI#60 ·
Maintainers usually reply within 1 day
-
enhancement good first issue
Difficulty 3/5 1-2 days Newbie friendliness 72/100
omlahore/RemoveMacAI#59 ·
Maintainers usually reply within 1 day
-
Difficulty 4/5 3-5 days Newbie friendliness 42/100
omlahore/RemoveMacAI#55 · 1 comment ·
Maintainers usually reply within 1 day
-
enhancement
Difficulty 3/5 1-2 days Newbie friendliness 58/100
omlahore/RemoveMacAI#54 · 1 comment ·
Maintainers usually reply within 1 day
All issues in omlahore/RemoveMacAI
Similar issues
-
Review backlog (style): Test helper still reads stdout after waitUntilExitPossibly taken @hivecommons-hive claimed this today. Openagent/reviewer help wanted hive/claimed-by-scanner stage/030-pr-in-progress
Difficulty 2/5 1-3 hours Newbie friendliness 77/100
hivecommons/hotshot#243 · 1 comment ·
Maintainers usually reply within 1 day
-
good first issue
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 67/100
Maintainers usually reply within 1 day
-
bug tests
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
jonathanong/no-mistakes#2261 ·
Maintainers usually reply within 1 day
-
addition
Difficulty 1/5 Under an hour Newbie friendliness 82/100
jaywcjlove/awesome-mac#3621 ·
Maintainers usually reply within 1 day