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

Consistency in opcode casing. #92

Open
tigh-latte opened this issue Dec 3, 2021 · 0 comments
Open

Consistency in opcode casing. #92

tigh-latte opened this issue Dec 3, 2021 · 0 comments
Labels
bug Something isn't working interpreter script interpreter

Comments

@tigh-latte
Copy link
Contributor

Currently through our code we have references to opcode and opCode, in both internal var names and error types.

This inconstency should be address before releasing v2, to prevent it become annoying and introducing the need for either accepting inconsistent behaviour, or releasing a breaking change.

I am personally going to argue for opcode and Opcode to be used, instead of opCode and OpCode, as in written English it is only one word and officially written as opcode.

@tigh-latte tigh-latte added the bug Something isn't working label Dec 3, 2021
@jadwahab jadwahab added the interpreter script interpreter label Dec 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working interpreter script interpreter
Projects
None yet
Development

No branches or pull requests

2 participants