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

S'assure que l'OS choisi pour installation est un nombre #6183

Merged

Conversation

philippemilink
Copy link
Member

Fix #5343

Vérifie que le caractère saisi pour sélectionner l'OS dans le script d'installation est bien un chiffre.

Contrôle qualité

make install-linux puis amusez-vous à saisir des caractères lors du choix de l'OS:

  • des caractères non-ASCII (é par exemple)
  • des caractères qui ne sont pas des chiffres
  • 0
  • un chiffre qui n'est pas proposé
  • un chiffre proposé

@philippemilink philippemilink added the hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest label Oct 2, 2021
@coveralls
Copy link

coveralls commented Oct 2, 2021

Coverage Status

Coverage remained the same at 86.619% when pulling 33e2b75 on philippemilink:check-number-select-os into 3259547 on zestedesavoir:dev.

Arnaud-D
Arnaud-D previously approved these changes Oct 3, 2021
Copy link
Contributor

@Arnaud-D Arnaud-D left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA OK ✔️

J'ai juste une petite suggestion typographique. ^^

Sinon tout fonctionne. Le comportement précédent était un peu erratique. Avec certains caractères, ça lançait l'installation en faisant un choix mystérieusement. Maintenant, c'est bien plus robuste.

scripts/install_zds.sh Outdated Show resolved Hide resolved
Co-authored-by: Arnaud-D <35631001+Arnaud-D@users.noreply.github.com>
@philippemilink philippemilink enabled auto-merge (squash) October 3, 2021 12:33
@philippemilink philippemilink merged commit 3ff576e into zestedesavoir:dev Oct 3, 2021
@philippemilink philippemilink deleted the check-number-select-os branch October 3, 2021 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[installation] Erreur choix de l'OS si on choisi l'OS "é"
3 participants