[ckan-changes] [ckan/ckan] 76222e: [#1673] Change testing standards saying that not a...

GitHub noreply at github.com
Thu May 1 16:17:03 UTC 2014


  Branch: refs/heads/master
  Home:   https://github.com/ckan/ckan
  Commit: 76222ea7eafeda684bae29bfece3495543e0a2ec
      https://github.com/ckan/ckan/commit/76222ea7eafeda684bae29bfece3495543e0a2ec
  Author: Vitor Baptista <vitor at vitorbaptista.com>
  Date:   2014-04-24 (Thu, 24 Apr 2014)

  Changed paths:
    M doc/contributing/testing.rst

  Log Message:
  -----------
  [#1673] Change testing standards saying that not all methods need docstrings

If we follow the other coding standards, like making small tests, naming, etc.,
they should all be quite understandable just from the test name. So we leave
the option to add docstrings to the developer that's creating it, if she can't
make it simple enough not to need it.


  Commit: 0074a9d01773781e4f746d93772f88b2b3e55cb6
      https://github.com/ckan/ckan/commit/0074a9d01773781e4f746d93772f88b2b3e55cb6
  Author: Ian Ward <ian at excess.org>
  Date:   2014-05-01 (Thu, 01 May 2014)

  Changed paths:
    M doc/contributing/testing.rst

  Log Message:
  -----------
  Merge remote-tracking branch 'origin/1673-not-all-test-methods-need-docstrings'


Compare: https://github.com/ckan/ckan/compare/bf3b4d71090a...0074a9d01773


More information about the ckan-changes mailing list