-
Notifications
You must be signed in to change notification settings - Fork 124
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
Enum will help with problems #159
Comments
@cesarjhony Thank you for suggesting this improvement. I'll take a look to see what can be done to improve things. |
Ive used HIGH / LOW in my work, as exported here: Line 343 in 07d36e2
while the TS version does something a bit different: Line 1 in 07d36e2
I would suggest that the js->d.ts is a stub solution at this point and there the Anything to aid in the correctness / clearness / ease of use would be interesting as many libraries "export" this type of |
My usage was just: I don't know if will better to be grouped style like: Thank you very much for appreciate my opinion 😊, but I have no better ideas. |
I dispend many of time searching bugs in my code, and when I found was a problem like:
pin.writeSync(true); //not work
pin.writeSync(1); //work fine
pin.writeSync(Gpio.HIGH); //work fine, but ENUM is not a Standard in your lib.
But, I and the most of devs use autocomplete, like a VS Code. With the STATIC or ENUM constants might save the day, the assistents find there and show us the options.
Please add all options like a static.
Your framework is amazing, but it still may be better.
The text was updated successfully, but these errors were encountered: