Write contribution guidelines about axelrod HOT 12 CLOSED

This should wait till #100 and #99 are taken care of. See #98.

from axelrod.

and meaningful variable names (please)!!

from axelrod.

Yup, great will add that also.

I'm thinking of making pep8 be part of the testing suite on Travis... Any reason not to? (330 things fail pep8 at the moment)

from axelrod.

I hate to play the gadly again, but there is no real reason for this and it would be a distraction. Do you really need code that is squeaky clean? Style guides are just that, guides. Not rules. See this part of PEP8 itself:
https://www.python.org/dev/peps/pep-0008/#a-foolish-consistency-is-the-hobgoblin-of-little-minds

from axelrod.

I tend to agree with @langner

Style guidelines are nice to have, but can end up causing more problems than they solve.

For example, some of the PEP8 errors I've fixed along the way have been simply that a line is more than 79 characters. It simply isn't worth the effort to go through the codebase to find and correct them.

from axelrod.

To be clear - I think the contribution guidelines should include PEP, but there's no point including compliance as part of the test suite.

from axelrod.

Cool, that sounds good to me, will leave the PEP testing out of Travis but
will include something about PEP in the guidelines. I'll throw them on a
separate branch (might get to it today) and ping you guys to see if you
think what I've written is good :)

On Mon, 9 Mar 2015 15:01 Owen Campbell [email protected] wrote:

I tend to agree with @langner https://github.com/langner

Style guidelines are nice to have, but can end up causing more problems
than they solve.

For example, some of the PEP8 errors I've fixed along the way have been
simply that a line is more than 79 characters. It simply isn't worth the
effort to go through the codebase to find and correct them.

—
Reply to this email directly or view it on GitHub
https://github.com/drvinceknight/Axelrod/issues/101#issuecomment-77869555
.

from axelrod.

Hey @langner and @meatballs , I've just pushed this as contribution guidelines: https://github.com/drvinceknight/Axelrod/blob/c-guidelines/docs/contributing.rst

Let me know what you think. If I don't hear anything I'll assume it's ok (kind of a guidelines/tutorial ish thing).

Once that's merged in to master I'm going to try and go to down on documentation... (by the way: I'm writing this using wifi on a plane! I can't get over how cool that is... I'm in the future!)

from axelrod.

I skimmed over it and it seems fine.

from axelrod.

👍

from axelrod.

Sorry - been down with a dose of food poisoning! Looks good to me too.

from axelrod.

Cool, thanks both. That is now merged in to master.

from axelrod.