search

rockneurotiko / ex_gram

Telegram Bot API low level API and framework
182 stars 26 forks source link

Add styler, update deps #143

Closed prtngn closed 8 months ago

prtngn commented 8 months ago

Add Adobe styler, update some deps and some fixes in DSL

rockneurotiko commented 8 months ago

I tried the styler in local and it does change much more things than the changes in the PR

I'm not a huge fan of Styler, there are times that I decide to not go with conventional style just for legibility, and not being able to ignore or anything it's bad, but I understand the decision.

Anyway, I don't mind adding it on this project, but instead of mixing different things on the same PR, I would appreciate one with the deps updates + styler (applied), and another with the change in DSL, one of them is major since it changes the spec of a public method of the DSL that potentially people are using

prtngn commented 8 months ago

Ok, thank for you reply. I make two different MRs.