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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Snake Case #25

Closed
Skillz4Killz opened this issue May 15, 2020 · 2 comments 路 Fixed by #27
Closed

Snake Case #25

Skillz4Killz opened this issue May 15, 2020 · 2 comments 路 Fixed by #27
Assignees
Labels
w-question Further information is requested

Comments

@Skillz4Killz
Copy link
Contributor

Skillz4Killz commented May 15, 2020

Hello 馃憢

A few users have noted that they see some odd instances of snake case in the library and I wanted to take a minute to address it.

When I started this library my goal was to learn different things and try different things. Most of what I did in this lib is for me to experiment and learn. I realized how terrible it was to work with snake_case. This became quite problematic to use in the long run so I opted for camelCase. If I missed any that are still in snake_case I will 100% be correcting them.

If you see any please leave them in a comment below, I will keep this issue pinned for a while.

Note: Since this will be changing the public API for some methods/props, we will be making this as v2 branch once all of them have been fixed.

@resynth1943
Copy link
Contributor

So are we just camelCasing everything?

@Skillz4Killz
Copy link
Contributor Author

Skillz4Killz commented May 18, 2020

@resynth1943 Yes snake case became way too difficult to use for me. The more i coded with it the less i enjoyed it and i just never got used to coding it that way. It also makes having to type another character _ which isn't nice especially with deno fmt which only allows 80 characters per line making it harder to read code

I did most of them already before 1.0 but it seems i forgot some

@Skillz4Killz Skillz4Killz unpinned this issue May 19, 2020
@resynth1943 resynth1943 removed their assignment Oct 31, 2020
TriForMine referenced this issue in TriForMine/discordeno May 1, 2021
Added support for mentioning bot as a prefix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
w-question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants