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

bug: snip-2 and snip-3 should be updated to use snake case #25

Closed
moodysalem opened this issue Jun 18, 2023 · 2 comments
Closed

bug: snip-2 and snip-3 should be updated to use snake case #25

moodysalem opened this issue Jun 18, 2023 · 2 comments
Labels
bug Something isn't working stale

Comments

@moodysalem
Copy link
Contributor

snake case is more typical for cairo code

@moodysalem moodysalem added the bug Something isn't working label Jun 18, 2023
@github-actions
Copy link

There hasn't been any activity on this issue recently, and in order to prioritize active issues, it will be marked as stale.
Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by leaving a 👍
Because this issue is marked as stale, it will be closed and locked in 7 days if no further activity occurs.
Thank you for your contributions!

@github-actions github-actions bot added the stale label Jul 19, 2023
@moodysalem
Copy link
Contributor Author

this is quite hard to change with all the existing erc20 camel case impls that do not also impl snake case so i would now prefer just keeping camel case from existing standards

@moodysalem moodysalem closed this as not planned Won't fix, can't repro, duplicate, stale Jul 19, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Aug 19, 2023
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working stale
Projects
None yet
Development

No branches or pull requests

1 participant