convexengineering / gplibrary

Useful subsystem models
MIT License
10 stars 11 forks source link

PyLint for models #101

Open galbramc opened 7 years ago

galbramc commented 7 years ago

@bqpd and @whoburg, should all the research models also use pylint?

bqpd commented 7 years ago

Probably, though I wouldn't make it email everyone just yet... Gentle onboarding would be great, though!

On Dec 30, 2016 17:52, "Marshall Galbraith" notifications@github.com wrote:

Should all the research models also use pylint?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/hoburg/gpkit-models/issues/101, or mute the thread https://github.com/notifications/unsubscribe-auth/ABagGNtrBcGdBGwIfr7bCBpBvK69haIZks5rNbVZgaJpZM4LYd2O .

bqpd commented 7 years ago

Using the pylintrc and settings from gpkit/examples would be the way to go here

On Dec 31, 2016 11:58, "Edward Burnell" eburn@mit.edu wrote:

Probably, though I wouldn't make it email everyone just yet... Gentle onboarding would be great, though!

On Dec 30, 2016 17:52, "Marshall Galbraith" notifications@github.com wrote:

Should all the research models also use pylint?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/hoburg/gpkit-models/issues/101, or mute the thread https://github.com/notifications/unsubscribe-auth/ABagGNtrBcGdBGwIfr7bCBpBvK69haIZks5rNbVZgaJpZM4LYd2O .