[Feature] Επιλογή σταθμού σε περισσότερες θέσεις από 9 - #113
[Feature] Επιλογή σταθμού σε περισσότερες θέσεις από 9#113kyriakosschwarz wants to merge 6 commits into
Conversation
Sorry, something went wrong.
There was a problem hiding this comment.
It was originally supposed to be a "quick start" feature, which explains the 1-9 implementation. Nevertheless, I am totally in favor of such a change, as long as it is working correctly. 👍🏼
+([1-9])) does not take zeros into account, so shelldio 10 will lead to "wrong selection" response.
By the way, we also need to make sure that extended glob (shopt -s extglob) is available on all platforms, including different Linux flavors and macOS.
Sorry, something went wrong.
| while [ "$1" != "" ]; do | ||
| case $1 in | ||
| [1-9]) | ||
| +([1-9])) |
There was a problem hiding this comment.
What about 10, 20, 30 etc?
Sorry, something went wrong.
There was a problem hiding this comment.
Στην δοκιμή μου, έβαλα 15 αγαπημένα. έτρεξα shelldio 14 και έβγαλε Λάθος επιλογή
Sorry, something went wrong.
|
Δοκιμές:
Άρα η λύση που προτείνει ο @kyriakosschwarz δουλεύει μια χαρά αρκεί να μην έχεις αγαπημένη λίστα. Οπότε δεν μπορεί να γίνει ακόμα merge μέχρι να γίνει η απαραίτητη διόρθωση. |
Sorry, something went wrong.
The integer shell parameter now can be arbitrarily large. Previously 1-9.