PyBindGen is not fully pep8 compliant

Bug #449315 reported by Jamie Kirkpatrick
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
PyBindGen
Confirmed
Low
Unassigned

Bug Description

Whilst it is stated that PyBindgen is pep8 compliant, this is not the case. pep8 specifies that lines should not be longer than 80 lines, and there are a huge number in the project that are.

It would be a small change to move towards the standard and would lower the barrier for entry for new participants (silly I know but it really does help when everything is consistent).

If the author is not against this, I would be happy to take it on.

Revision history for this message
Gustavo Carneiro (gjc) wrote :

I am not against it. PEP 8 is PEP 8. I may not like everything in it, but it's better to be consistent.
Patches welcome.

Revision history for this message
Jamie Kirkpatrick (jamiekp) wrote :

OK, I will submit something when I get a chance...it will be a fairly big patch!

Revision history for this message
Gustavo Carneiro (gjc) wrote :

Feel free to use launchpad to its full potential, i.e. create a bazaar clone of the repo, publish in launchpad, make your modifications, and "propose a merge" of your branch into the main one. It's not so complicated as it sounds.

Revision history for this message
Jamie Kirkpatrick (jamiekp) wrote : Re: [Bug 449315] Re: PyBindGen is not fully pep8 compliant

Don't worry, I'm a GitHub user so the procedure is not new on me :)
Good chance to play with Bazaar too...

2009/10/12 Gustavo Carneiro <email address hidden>:
> Feel free to use launchpad to its full potential, i.e. create a bazaar
> clone of the repo, publish in launchpad, make your modifications, and
> "propose a merge" of your branch into the main one.  It's not so
> complicated as it sounds.
>
> --
> PyBindGen is not fully pep8 compliant
> https://bugs.launchpad.net/bugs/449315
> You received this bug notification because you are a direct subscriber
> of the bug.
>
> Status in Python Bindings Generator: New
>
> Bug description:
> Whilst it is stated that PyBindgen is pep8 compliant, this is not the case.  pep8 specifies that lines should not be longer than 80 lines, and there are a huge number in the project that are.
>
> It would be a small change to move towards the standard and would lower the barrier for entry for new participants (silly I know but it really does help when everything is consistent).
>
> If the author is not against this, I would be happy to take it on.
>

--
Jamie Kirkpatrick
07818 422311

Gustavo Carneiro (gjc)
Changed in pybindgen:
status: New → Confirmed
importance: Undecided → Low
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.