ok, those are fixed now. (readonly, uneeded import and too long lines). testsuite is still ok and here is the new make lint output:
$ make lint
utilities/shhh.py PYTHONPATH= python2.5 bootstrap.py\ --ez_setup-source=ez_setup.py \
--download-base=download-cache/dist --eggs=eggs
Enter passphrase for key '/home/ubuntu/.ssh/id_rsa':
Enter passphrase for key '/home/ubuntu/.ssh/id_rsa':
= Launchpad lint =
Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.
lib/lp/registry/interfaces/person.py
53: [F0401] Unable to import 'lazr.enum' (No module named enum)
54: [F0401] Unable to import 'lazr.lifecycle.snapshot' (No module named lifecycle)
55: [F0401] Unable to import 'lazr.restful.interface' (No module named restful)
56: [F0401] Unable to import 'lazr.restful.declarations' (No module named restful)
63: [F0401] Unable to import 'lazr.restful.fields' (No module named restful)
410: [E1002, PersonNameField._validate] Use super on an old style class
1404: [C0322, IPersonEditRestricted.addMember] Operator not preceded by a space
status=copy_field(ITeamMembership['status']),
^
comment=Text(required=False))
@export_write_operation()
def addMember(person, reviewer, status=TeamMembershipStatus.APPROVED,
comment=None, force_team_add=False,
may_subscribe_to_list=True):
1445: [C0322, IPersonEditRestricted.acceptInvitationToBeMemberOf] Operator not preceded by a space
comment=Text())
^
@export_write_operation()
def acceptInvitationToBeMemberOf(team, comment):
1457: [C0322, IPersonEditRestricted.declineInvitationToBeMemberOf] Operator not preceded by a space
comment=Text())
^
@export_write_operation()
def declineInvitationToBeMemberOf(team, comment):
1755: [C0322, IPersonSet.newTeam] Operator not preceded by a space
defaultmembershipperiod='default_membership_period',
^
defaultrenewalperiod='default_renewal_period')
@operation_parameters(
subscriptionpolicy=Choice(
title=_('Subscription policy'), vocabulary=TeamSubscriptionPolicy,
required=False, default=TeamSubscriptionPolicy.MODERATED))
@export_factory_operation(
ITeam, ['name', 'displayname', 'teamdescription',
'defaultmembershipperiod', 'defaultrenewalperiod'])
def newTeam(teamowner, name, displayname, teamdescription=None,
subscriptionpolicy=TeamSubscriptionPolicy.MODERATED,
defaultmembershipperiod=None, defaultrenewalperiod=None):
1824: [C0322, IPersonSet.findPerson] Operator not preceded by a space
created_after=Datetime(
^
title=_("Created after"), required=False),
created_before=Datetime(
title=_("Created before"), required=False),
)
@operation_returns_collection_of(IPerson)
@export_read_operation()
def findPerson(text="", exclude_inactive_accounts=True,
must_have_email=False,
created_after=None, created_before=None):
lib/lp/registry/interfaces/ssh.py
18: [F0401] Unable to import 'lazr.enum' (No module named enum)
19: [F0401] Unable to import 'lazr.restful.declarations' (No module named restful)
The changes are independent from jml's change. So if you can merge it, it would rocks! Thanks a lot!
ok, those are fixed now. (readonly, uneeded import and too long lines). testsuite is still ok and here is the new make lint output:
$ make lint
--ez_ setup-source= ez_setup. py \ base=download- cache/dist --eggs=eggs ubuntu/ .ssh/id_ rsa': ubuntu/ .ssh/id_ rsa':
utilities/shhh.py PYTHONPATH= python2.5 bootstrap.py\
--download-
Enter passphrase for key '/home/
Enter passphrase for key '/home/
= Launchpad lint =
Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.
Linting changed files: registry/ browser/ configure. zcml registry/ browser/ tests/test_ sshkey. py registry/ interfaces/ person. py registry/ interfaces/ ssh.py registry/ stories/ webservice/ xx-person. txt
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
== Pylint notices ==
lib/lp/ registry/ interfaces/ person. py .snapshot' (No module named lifecycle) interface' (No module named restful) declarations' (No module named restful) fields' (No module named restful) ._validate] Use super on an old style class ricted. addMember] Operator not preceded by a space copy_field( ITeamMembership ['status' ]), Text(required= False)) write_operation () TeamMembershipS tatus.APPROVED, add=False, subscribe_ to_list= True): ricted. acceptInvitatio nToBeMemberOf] Operator not preceded by a space write_operation () nToBeMemberOf( team, comment): ricted. declineInvitati onToBeMemberOf] Operator not preceded by a space write_operation () onToBeMemberOf( team, comment): ershipperiod= 'default_ membership_ period' , walperiod= 'default_ renewal_ period' ) parameters( npolicy= Choice( _('Subscription policy'), vocabulary= TeamSubscriptio nPolicy, TeamSubscriptio nPolicy. MODERATED) ) factory_ operation( bershipperiod' , 'defaultrenewal period' ]) =None, npolicy= TeamSubscriptio nPolicy. MODERATED, ershipperiod= None, defaultrenewalp eriod=None) : findPerson] Operator not preceded by a space after=Datetime( _("Created after"), required=False), before= Datetime( _("Created before"), required=False), returns_ collection_ of(IPerson) read_operation( ) inactive_ accounts= True, have_email= False, after=None, created_ before= None):
53: [F0401] Unable to import 'lazr.enum' (No module named enum)
54: [F0401] Unable to import 'lazr.lifecycle
55: [F0401] Unable to import 'lazr.restful.
56: [F0401] Unable to import 'lazr.restful.
63: [F0401] Unable to import 'lazr.restful.
410: [E1002, PersonNameField
1404: [C0322, IPersonEditRest
status=
^
comment=
@export_
def addMember(person, reviewer, status=
comment=None, force_team_
may_
1445: [C0322, IPersonEditRest
comment=Text())
^
@export_
def acceptInvitatio
1457: [C0322, IPersonEditRest
comment=Text())
^
@export_
def declineInvitati
1755: [C0322, IPersonSet.newTeam] Operator not preceded by a space
defaultmemb
^
defaultrene
@operation_
subscriptio
title=
required=False, default=
@export_
ITeam, ['name', 'displayname', 'teamdescription',
'defaultmem
def newTeam(teamowner, name, displayname, teamdescription
subscriptio
defaultmemb
1824: [C0322, IPersonSet.
created_
^
title=
created_
title=
)
@operation_
@export_
def findPerson(text="", exclude_
must_
created_
lib/lp/ registry/ interfaces/ ssh.py declarations' (No module named restful)
18: [F0401] Unable to import 'lazr.enum' (No module named enum)
19: [F0401] Unable to import 'lazr.restful.
The changes are independent from jml's change. So if you can merge it, it would rocks! Thanks a lot!