Update Best Practices in CONTRIBUTING.md

[cedricconol]

Added two items under Best Practices

7. Maximum docstring line length should be 75 characters. This should be manually done as `black` formatting does not enforce limits on docstring line length.
8. Multiline code in docstring should be enclosed in parenthesis where each line after the first starting with an ellipsis (`...`), like so:

>>> s = (
...     s.pipe(hero.clean)
...      .pipe(hero.tokenize)
...      .pipe(hero.term_frequency)
...      .pipe(hero.flatten)
... )

[jbesomi]

This should be manually done... -> done manually ...

[jbesomi]

Maybe we can replace this sentence with:

"In docstring test, on long pipe, consider enclosing the code in parenthesis like so"

It's a bit shorter and clearer

[cedricconol]

Noted, Jonathan. However, I think you meant "In docstring examples.." rather than "In docstring test.." right?

[jbesomi]

Yes, sorry!

[jbesomi]

Thank you @cedricconol, merged! (sorry for my late replay, I'm doing university exams right now)

[cedricconol]

@jbesomi sorry, I think i was not clear enough. I actually have not implemented your suggestion to use "In docstring test.." yet, I was waiting for your confirmation.

is there a way to roll back the merge? or should I just open a new PR for that?

[jbesomi]

Hi Cedric, oh I see, no problem: just open a new PR and make the new changes.
Thanks!

[cedricconol]

here's the new PR #154