Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #6954 (closed)

Opened 13 years ago

Closed 9 years ago

Python Code style template

Reported by: jburel Owned by:
Priority: minor Milestone: Future
Component: General Version: 5.1.2
Keywords: n.a. Cc: python-team@…
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description

Provide a code style template to use while writing Python code

Change History (7)

comment:1 Changed 13 years ago by jburel

  • Milestone changed from Unscheduled to OME-5.0

comment:2 Changed 13 years ago by jburel

Style guide for Python code that the Cell Profiler team attempt to follow
http://www.python.org/dev/peps/pep-0008/

comment:3 Changed 12 years ago by wmoore

Just tried a pep8 bundle for Text-Mate: https://github.com/ppierre/python-pep8-tmbundle

Summary report for gateway/__init__.py

 

   7      E111 : indentation is not a multiple of four
   7      E201 : whitespace after '('
   5      E202 : whitespace before ')'
 335      E211 : whitespace before '('
   1      E222 : multiple spaces after operator
  65      E231 : missing whitespace after ':'
  54      E301 : expected 1 blank line, found 2
  53      E302 : expected 2 blank lines, found 1
   2      E401 : multiple imports on one line
 482      E501 : line too long (102 characters)
   2      E701 : multiple statements on one line (colon)
  10      E702 : multiple statements on one line (semicolon)
 804      W291 : trailing whitespace
   4      W601 : .has_key() is deprecated, use 'in'

comment:4 Changed 12 years ago by wmoore

  • Cc python-team@… added; jmoore cxallan removed

comment:5 Changed 12 years ago by jburel

  • Milestone changed from OMERO-Beta4.4 to Future

No decision made on Python front. pushing to Future.

comment:6 Changed 12 years ago by jmoore

Referencing ticket #6952 has changed sprint.

comment:7 Changed 9 years ago by sbesson

  • Resolution set to fixed
  • Status changed from new to closed
  • Version set to 5.1.2

Python code is now fully flake8'ed.

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.67239 sec.)

We're Hiring!