libturpial needs your help!

15 views
Skip to first unread message

Milton Mazzarri

unread,
Mar 10, 2014, 2:12:20 PM3/10/14
to turpi...@googlegroups.com
Hi everybody,

Do you want to begin to contribute into libturpial codebase but you don't know
where to start?, that's ok, it also happens to me sometimes, but today, the
reality is that we need your help to fix some errors reported by our style
checker (flake8), this errors are:

# E126 continuation line over-indented for hanging indent
# E128 continuation line under-indented for visual indent
# E231 missing whitespace after ':'
# E251 unexpected spaces around keyword / parameter equals
# E261 at least two spaces before inline comment
# E301 expected 1 blank line
# E302 expected 2 blank lines
# E303 too many blank lines
# E501 line too long
# E711 comparison to None should be 'if cond is not None:'
# E712 comparison to False should be 'if cond is False:' or 'if not cond:'
# F401 imported but unused
# F403 unable to detect undefined names
# F821 undefined name
# F841 local variable is assigned to but never used
# F999 syntax error in doctest
# W291 trailing whitespace
# W391 blank line at end of file
# W601 .has_key() is deprecated, use 'in'

As you see, some errors are really easy to fix but are too many for us, so
please, we desperately need your help, help us!

The following is how we expect the community can contribute code into
libturpial via Pull Requests.

1) If you don't have a Github[0] account, please create one first.

2) Fork our project

3) Install Git[1], after that you should setup your name and email:

$ git config --global user.name "Your Name, not your Github nickname"
$ git config --global user.email "y...@example.com"

4) Set your local repository

git clone https://github.com/your_github_nickname/libturpial.git

5) Set 'satanas/libturpial' as your 'upstream' remote, this basically tells Git
that your are referencing libturpial's repository as your main source.

$ git remote add upstream https://github.com/satanas/libturpial.git

5.1) Update your local copy

$ git fetch upstream

6) Verify that no one else has been working on the same bug, you can check that
in our issues[2] list, in this list you can also check Pull Requests pending
for the BDFL approval.

7) Working on a bug

7.1) In the first place, install tox

$ pip install tox

7.2) Then, create a branch that identifies the bug that you will begin to work
on:

$ git checkout -b E231 upstream/development

In this case we are working on the bugs of the type: E231 (as indicate by
flake8)

7.3) Make some local changes and commit, repeat.

7.3.1) Delete the error code that you will begin to work from the ignore list
located at the 'flake8' section in the 'tox.ini' file (located at the root of
the project)

Example:
# Original list
ignore = E126,E128,E231,E251,E261,E301,...

# After we decide to work in the E231 error
ignore = E126,E128,E251,E261,E301,...

7.3.2) Execute "tox -e docs" to check the current errors.

7.3.3) Fix, fix, fix...

7.3.4) Commit your changes

$ git commit -m "Fixed errors 'E231' according to flake8."

7.4) In the case that you fixed all errors of the same type, please delete the
corresponding line in the tox.ini file (located at the root)

7.4.1) Don't forget to commit that.

8) Publish your work

$ git push origin E231

Please, adjust the name 'E231' to something more appropiate in your case.

9) Create a Pull Request on Github

10) Enjoy your work! :-)

You can find more details on our guide "How to write code"[3], also, in case of
doubt don't hesitate to reach us.

Thanks in advance,

[0] https://github.com
[1] http://git-scm.com/download
[2] https://github.com/satanas/libturpial/issues
[3] http://libturpial.readthedocs.org/en/latest/dev/write.html

--
Milton Mazzarri, a.k.a. [milmazz]
Usuario de GNU/Linux: #369158
http://milmazz.com.ve

Sebastián Magrí

unread,
Mar 10, 2014, 6:09:34 PM3/10/14
to turpial-dev
Thanks Milton!

This should be promoted to the wiki.

Regards,


--
Has recibido este mensaje porque estás suscrito al grupo "Turpial-Dev" de Grupos de Google.
Para anular la suscripción a este grupo y dejar de recibir sus correos electrónicos, envía un correo electrónico a turpial-dev...@googlegroups.com.
Para obtener más opciones, visita https://groups.google.com/d/optout.



--
Sebastián Ramírez Magrí

Wil Alvarez

unread,
Mar 10, 2014, 6:20:03 PM3/10/14
to turpi...@googlegroups.com

Milton, thanks for this awesome guide. I hope we can find more hands to code :)

On Mar 10, 2014 7:09 PM, "Sebastián Magrí" <sebas...@gmail.com> wrote:
>
> Thanks Milton!
>
> This should be promoted to the wiki.
>

Yes! That's exactly what I was thinking right now Sebastián. I'm going to create that page on the wiki tonight at home.

Thanks guys

Reply all
Reply to author
Forward
0 new messages