Closed Jigarvarma2005 closed 2 years ago
@Jigarvarma2005, Thanks for updating this PR.
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
@Jigarvarma2005 try this
@Jigarvarma2005, Thanks for updating this PR.
- In the file
userge/plugins/tools/executor.py
:Line 167:16: E127 continuation line over-indented for visual indent Line 169:14: E126 continuation line over-indented for hanging indent
Comment last updated at 2021-04-26 15:29:30 UTC
Why😬😬
@Jigarvarma2005, Thanks for updating this PR.
- In the file
userge/plugins/tools/executor.py
:Line 170:5: E124 closing bracket does not match visual indentation
Comment last updated at 2021-04-26 15:42:19 UTC
Why pep8 always suck🤨
🐛 🐛 🐛
good, but dangerous
good, but dangerous
Why?😬🤔
files may contain dangerous codes like delete_account, spam, ...
. if anyone .eval
that, boom ...
files may contain dangerous codes like
delete_account, spam, ...
. if anyone.eval
that, boom ...
😬🤨😑
files may contain dangerous codes like
delete_account, spam, ...
. if anyone.eval
that, boom ...
Close this PR🤗
@Jigarvarma2005 lets wait for others opinions.
@Jigarvarma2005 lets wait for others opinions.
OK
seems good now.
@Jigarvarma2005, Thanks for updating this PR.
- In the file
userge/plugins/tools/executor.py
:Line 170:5: E124 closing bracket does not match visual indentation
Comment last updated at 2021-04-26 15:42:19 UTC
Why pep8 always suck🤨
Devs should just remove style checks from Deepsource issue config. It's useless and annoying. Using a transformer like autopep8
is rather smarter choice if style really matters.
@Jigarvarma2005, Thanks for updating this PR.
- In the file
userge/plugins/tools/executor.py
:Line 170:5: E124 closing bracket does not match visual indentation
Comment last updated at 2021-04-26 15:42:19 UTC
Why pep8 always suckraised_eyebrow
Devs should just remove style checks from Deepsource issue config. It's useless and annoying. Using a transformer like
autopep8
is rather smarter choice if style really matters.
its cool somehow (deepsource) :D
@rking32 auto transformers are way cooler ~ https://deepsource.io/docs/transformer/autopep8
This is a massive security loophole, and we don't need to be giving users an extra way to shoot themselves in the foot. I vote this be immediately closed and not merged.
This is a massive security loophole, and we don't need to be giving users an extra way to shoot themselves in the foot. I vote this be immediately closed and not merged.
Bruh
🤨
Will open it again, if need🤔
When in telegram we paste code most of times the syntax changes and we need to correct them there, so this