Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PE-D][Tester B] Help command accepts additional parameters #181

Closed
soc-pe-bot opened this issue Apr 5, 2024 · 3 comments · Fixed by #191
Closed

[PE-D][Tester B] Help command accepts additional parameters #181

soc-pe-bot opened this issue Apr 5, 2024 · 3 comments · Fixed by #191
Assignees
Labels
bug.Deferred Not allowed under v1.4, mention in relevant documents (UG/DG) if necessary severity.Low Bugs that are unlikely to affect normal operations

Comments

@soc-pe-bot
Copy link

Not sure if this is a bug exactly but the help command followed by a space and additional strings will still bring up the help window instead of throwing an error.

image.png


Labels: type.FunctionalityBug severity.Low
original: quelinxiao/ped#4

@gerteck
Copy link

gerteck commented Apr 6, 2024

Not really a bug imo, added feature ftw

@chaaaaun chaaaaun added priority.Low severity.Low Bugs that are unlikely to affect normal operations and removed priority.Low labels Apr 8, 2024
@chaaaaun
Copy link

chaaaaun commented Apr 8, 2024

ig to be explicit we can disallow, should be an easy fix

@CJerrong CJerrong added the bug.Deferred Not allowed under v1.4, mention in relevant documents (UG/DG) if necessary label Apr 9, 2024
@gerteck
Copy link

gerteck commented Apr 9, 2024

Handling extraneous inputs (e.g., extra parameters, repeated parameters etc.) in commands:
The command 'forgiving' these extraneous inputs (i.e., giving an output same as or similar to if those inputs are not present) is not incorrect. You can mention in the UG that such inputs will be ignored. AB3 already does a similar thing for some commands. Any special handling of such inputs can be left as a future enhancement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug.Deferred Not allowed under v1.4, mention in relevant documents (UG/DG) if necessary severity.Low Bugs that are unlikely to affect normal operations
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants