search

cms-sw / cmssw

CMS Offline Software
http://cms-sw.github.io/
Apache License 2.0
1.09k stars 4.33k forks source link

code-checks/format for python #30775

Open slava77 opened 4 years ago

slava77 commented 4 years ago

now that we are regularly formatting c++, it would be nice to add something similar for python

slava77 commented 4 years ago

assign core

cmsbuild commented 4 years ago

New categories assigned: core

@Dr15Jones,@smuzaffar,@makortel you have been requested to review this Pull request/Issue and eventually sign? Thanks

cmsbuild commented 4 years ago

A new Issue was created by @slava77 Slava Krutelyov.

@Dr15Jones, @silviodonato, @dpiparo, @smuzaffar, @makortel can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

slava77 commented 4 years ago

some online pointers (although I have a rather limited experience with python)

https://google.github.io/styleguide/pyguide.html https://github.com/google/yapf

slava77 commented 4 years ago

ping

davidlange6 commented 4 years ago

There are a number of options - autopep8 is available in CMSSW. I’ve Also used black for example.

On 14 Aug 2020, at 15:48, Slava Krutelyov notifications@github.com wrote:



ping

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/cms-sw/cmssw/issues/30775#issuecomment-674083394, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABGPFQZQ7WSPE2EJEFPOZW3SAU6ANANCNFSM4O6HKMAQ.

tvami commented 3 years ago

@smuzaffar is this among the plans? I think it would be very nice!

smuzaffar commented 3 years ago

yes it is also on my to do list :-)