Merge lp://staging/~nueces/lalita/support-private-channels into lp://staging/lalita

Proposed by nueces
Status: Needs review
Proposed branch: lp://staging/~nueces/lalita/support-private-channels
Merge into: lp://staging/lalita
Diff against target: 62 lines (+35/-3)
2 files modified
lalita.cfg.sample (+28/-1)
lalita/ircbot.py (+7/-2)
To merge this branch: bzr merge lp://staging/~nueces/lalita/support-private-channels
Reviewer Review Type Date Requested Status
Facundo Batista Needs Fixing
Review via email: mp+20866@code.staging.launchpad.net

Description of the change

This branch provide support to joining private channels that are protected by password.

To post a comment you must log in.
Revision history for this message
Facundo Batista (facundo) wrote :

This looks nice! However, some details:

- point at the end of the docstring

- remove blank in the ".get (...)"

And what it mandates a need fixing here: You need to provide test cases for this new feature.

Thank you very much!!

review: Needs Fixing

Unmerged revisions

138. By nueces

merge to sync with trunk

137. By nueces

Added support to joining private channels that are protected by password

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: