Skip to content

Translated isAlpha() #32

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

Merged
merged 2 commits into from
Nov 2, 2017
Merged

Translated isAlpha() #32

merged 2 commits into from
Nov 2, 2017

Conversation

robsoncouto
Copy link
Contributor

There seems to be something wrong with the example code tough.
It seems to be testing is a variable is alpha, but says to the user that is is alphanumeric.
May need fixing.

There seems to be something wrong with the example code tough.
It seems to be testing is a variable is alpha, but says to the user that is is alphanumeric.
May need fixing
@smellai
Copy link
Contributor

smellai commented Nov 2, 2017

you are right @robsoncouto, I opened an issue about that in the -en repo, you can fix it here in pt, thanks!

@robsoncouto
Copy link
Contributor Author

No problem :)
I noticed there is something similar in https://cdn.arduino.cc/reference/en/language/functions/characters/isdigit/
The comment say "tests if this is a letter" when it tests if the variable is a number. I fixed that one already here, as the problem was in the comment, not the code itself.
There may be other bugs like this in the other character functions I have not translated yet.

@smellai
Copy link
Contributor

smellai commented Nov 2, 2017

Fixed here: arduino/reference-en@292bcc0

Please check the last commits in reference-en, we fixed many broken links. If, when you translate a page, you could check if all the links in the page are ok, it would be really great. Thanks so much

@smellai smellai merged commit b224882 into arduino:master Nov 2, 2017
@robsoncouto
Copy link
Contributor Author

Hi @smellai . I find it quite hard to keep focused in the translation and fix the broken links at the same time. For now I will keep up the translation and add an issue for the broken links.

@robsoncouto robsoncouto deleted the patch-27 branch March 1, 2019 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants