search

sphericalpm / lektorium

A pragmatic web content management solution for those with way too many little sites
MIT License
16 stars 3 forks source link

#22: Update contributing file #27

Closed mjecov closed 5 years ago

mjecov commented 5 years ago

PR #22

katridi commented 5 years ago

Cool, what about this https://github.com/sphericalpm/lektorium/pull/21#discussion_r306811980 Pytest naming Check this out, please. Should we mention this?

mjecov commented 5 years ago

Cool, what about this #21 (comment) Pytest naming Check this out, please. Should we mention this?

Will see later. Maybe we will make additional test style guide. But in my view pytest naming is not about style. It is about pytest rules, so everybody who use it should use this naming to make his test findable.

katridi commented 5 years ago

Will see later. Maybe we will make additional test style guide. But in my view pytest naming is not about style. It is about pytest rules, so everybody who use it should use this naming to make his test findable.

Yep, it's good idea to make some style guides about tests