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.

Sign up for GitHub

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

Jump to bottom

[PE-D][Tester E] Invalid indexes giving different error messages for 'delete' command #171

Open
soc-se-bot opened this issue Apr 5, 2024 · 1 comment
Open

[PE-D][Tester E] Invalid indexes giving different error messages for 'delete' command #171

soc-se-bot opened this issue Apr 5, 2024 · 1 comment
Labels
Planned Enhancement

Comments

@soc-se-bot
Copy link

image.png

delete -1 shows error message of invalid command format (though i know it states positive) but i would think it would be better to show the same error message as for delete 9 (out of range, e.g shown below)
Would be better to have invalid index message for negative index cases as well.

image.png

Labels: severity.VeryLow type.FunctionalityBug
original: yapxuanxuan/ped#7
@narwhalsilent narwhalsilent added this to the 1.4 milestone Apr 6, 2024
@xiaorui-ui
Copy link

xiaorui-ui commented Apr 7, 2024 •
edited
Loading

Type label is wrong, not realistic to fix

@xiaorui-ui xiaorui-ui added bug.won'tFix bug.message For descriptions or error messages and removed bug.invalid labels Apr 7, 2024
@xiaorui-ui xiaorui-ui reopened this Apr 9, 2024
@Joseph31416 Joseph31416 added Planned Enhancement and removed bug.message For descriptions or error messages bug.won'tFix labels Apr 9, 2024
@narwhalsilent narwhalsilent removed this from the v1.4 milestone Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Planned Enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@Joseph31416 @xiaorui-ui @soc-se-bot @narwhalsilent