Differences between revisions 1 and 4 (spanning 3 versions)
Revision 1 as of 2020-02-15 15:56:39
Size: 423
Editor: chapoton
Comment:
Revision 4 as of 2020-02-15 16:23:07
Size: 2847
Editor: chapoton
Comment:
Deletions are marked like this. Additions are marked like this.
Line 5: Line 5:
Check for the existence of a commit message for every commit.
Line 6: Line 8:

Try to check that coverage did increase.
Line 9: Line 13:
Look for the presence of non-ascii characters in python and cython files.

This detects the presence of the encoding line at the top of files.
Line 10: Line 18:

Check that doctest continuation use the correct syntax, namely `....:`
Line 13: Line 23:
Check that some bad latex code does not appear.

This means '\choose', '\over', '\atop', '\above', '\overwithdelims', '\atopwithdelims', '\abovewithdelims'.
Line 14: Line 28:

Check that print is using python 3 syntax. (TO BE REMOVED)
Line 17: Line 33:
Look for some wrong patterns in python or rst files.

0) xrange

1) .iterkeys, .itervalues, .iteritems

2) basestring

3) __nonzero__

These are not allowed in python 3. (TO BE REMOVED)
Line 18: Line 46:

Look for some wrong patterns in cython files.

0) "import six" and "from six import"
Line 21: Line 53:
Check that some python3 incompatible code does not appear. (TO BE REMOVED)

2) ifilter, imap, izip

3) raise statements

4) cmp

6) <>

7) <type '[a-z]*'> (no longer needed)

8) next

9) __metaclass__

10) except Exception, var

11) apply

12) sagenb
Line 22: Line 76:

Run pyflakes on the modified .py files.

This typically reports about unused variables or imports.

Sometimes it gives false-positive warnings.
Line 25: Line 85:
Run ``pycodestyle --select=...`` on the modified .py files.

Currently, the selected options ae W605, E401, E701, E702.

See [http://pycodestyle.pycqa.org/en/latest/intro.html#error-codes] for more information.
Line 26: Line 92:

Perform various check, mainly about blocks in the documentation.

1) correct syntax is .. SEEALSO::

2) TESTS and EXAMPLES should be plural, and NOTE singular

3) no :: after INPUT and OUTPUT blocks, only a single colon

4) no :: after REFERENCE blocks, only a single colon
    
5) no " :" at the end of lines, as the colon should not be preceded by a space
    
6) no "Returns" at the start of lines, but "Return"
Line 29: Line 109:
Look for the presence of triple colons `:::` or `: ::`.
Line 30: Line 112:

Look for the presence of badly formatted trac roles ``:trac:``, missing the initial colon.
Line 33: Line 117:
Try to decide if the startup time is getting worse.

This performs a statistical analysis.
Line 34: Line 122:

Count modules imported at startup, and compare to stored data to see if this has increased.
Line 37: Line 127:
Build the html documentation.

== docbuild_pdf ==

Build the pdf documentation.

This is not activated by default on the patchbot clients.
Line 38: Line 136:

Not clear ?

explanations on the patchbot plugins

commit_messages

Check for the existence of a commit message for every commit.

coverage

Try to check that coverage did increase.

non_ascii

Look for the presence of non-ascii characters in python and cython files.

This detects the presence of the encoding line at the top of files.

doctest_continuation

Check that doctest continuation use the correct syntax, namely ....:

foreign_latex

Check that some bad latex code does not appear.

This means '\choose', '\over', '\atop', '\above', '\overwithdelims', '\atopwithdelims', '\abovewithdelims'.

oldstyle_print

Check that print is using python 3 syntax. (TO BE REMOVED)

python3_py

Look for some wrong patterns in python or rst files.

0) xrange

1) .iterkeys, .itervalues, .iteritems

2) basestring

3) nonzero

These are not allowed in python 3. (TO BE REMOVED)

python3_pyx

Look for some wrong patterns in cython files.

0) "import six" and "from six import"

python3

Check that some python3 incompatible code does not appear. (TO BE REMOVED)

2) ifilter, imap, izip

3) raise statements

4) cmp

6) <>

7) <type '[a-z]*'> (no longer needed)

8) next

9) metaclass

10) except Exception, var

11) apply

12) sagenb

pyflakes

Run pyflakes on the modified .py files.

This typically reports about unused variables or imports.

Sometimes it gives false-positive warnings.

pycodestyle

Run pycodestyle --select=... on the modified .py files.

Currently, the selected options ae W605, E401, E701, E702.

See [http://pycodestyle.pycqa.org/en/latest/intro.html#error-codes] for more information.

blocks

Perform various check, mainly about blocks in the documentation.

1) correct syntax is .. SEEALSO::

2) TESTS and EXAMPLES should be plural, and NOTE singular

3) no :: after INPUT and OUTPUT blocks, only a single colon

4) no :: after REFERENCE blocks, only a single colon

5) no " :" at the end of lines, as the colon should not be preceded by a space

6) no "Returns" at the start of lines, but "Return"

triple_colon

Look for the presence of triple colons ::: or : ::.

Look for the presence of badly formatted trac roles :trac:, missing the initial colon.

startup_time

Try to decide if the startup time is getting worse.

This performs a statistical analysis.

startup_modules

Count modules imported at startup, and compare to stored data to see if this has increased.

docbuild

Build the html documentation.

docbuild_pdf

Build the pdf documentation.

This is not activated by default on the patchbot clients.

git_rev_list

Not clear ?