Merge lp://staging/~tsyrus/pastebinit/pastebinthat into lp://staging/pastebinit

Proposed by Terence Syrus
Status: Work in progress
Proposed branch: lp://staging/~tsyrus/pastebinit/pastebinthat
Merge into: lp://staging/pastebinit
Diff against target: 1419 lines (+639/-483)
21 files modified
pastebin.d/cxg.de.conf (+4/-7)
pastebin.d/dpaste.com.conf (+17/-0)
pastebin.d/fpaste.org.conf (+1/-1)
pastebin.d/lpaste.net.conf (+2/-3)
pastebin.d/p.defau.lt.conf (+2/-5)
pastebin.d/paste.debian.net.conf (+1/-1)
pastebin.d/paste.drizzle.org.conf (+6/-9)
pastebin.d/paste.kde.org.conf (+10/-14)
pastebin.d/paste.openstack.org.conf (+6/-9)
pastebin.d/paste.pound-python.org.conf (+1/-1)
pastebin.d/paste.ubuntu.com.conf (+1/-1)
pastebin.d/paste.ubuntu.org.cn.conf (+1/-1)
pastebin.d/paste2.org.conf (+1/-1)
pastebin.d/pastebin.com.conf (+4/-5)
pastebin.d/pastebin.mate-desktop.org.conf (+7/-9)
pastebin.d/pb.daviey.com.conf (+4/-5)
pastebin.d/slexy.org.conf (+2/-3)
pastebin.d/sprunge.us.conf (+3/-5)
pastebinit (+417/-398)
test.py (+135/-0)
test.sh (+14/-5)
To merge this branch: bzr merge lp://staging/~tsyrus/pastebinit/pastebinthat
Reviewer Review Type Date Requested Status
Pastebinit Developers Pending
Review via email: mp+243250@code.staging.launchpad.net

Description of the change

Complete overhaul of pastebinit:
* significantly cleaned up code for less coupling and more readability
* changed config to a more sensible structure:
** only posted parameters are in the format section now
** parameters that were previously hidden but listed in the format section now live in the pastebin section
* added unit tests
* merged in all branches from ~blueyed

To post a comment you must log in.
216. By Terence Syrus <email address hidden>

added unit tests

217. By Terence Syrus <email address hidden>

removed unused method

218. By Terence Syrus <email address hidden>

removed debug

Revision history for this message
Stéphane Graber (stgraber) wrote :

Hi,

I'm sorry I didn't see this before, as you most likely figured out by now, I'm not very actively involved anymore as pastebinit pretty much just works for me and has done so for years.

I'm certainly happy with work to make it less crufty and better tested, unfortunately it looks like the bunch of changes I applied today makes this fail to apply.

Any chance you can rebase this on current pastebinit. I'll try to be faster to respond this time! :)

Unmerged revisions

218. By Terence Syrus <email address hidden>

removed debug

217. By Terence Syrus <email address hidden>

removed unused method

216. By Terence Syrus <email address hidden>

added unit tests

215. By Terence Syrus <email address hidden>

major overhaul: tests + cleanup

214. By Terence Syrus <email address hidden>

major cleanup
also, updated pastebin.mate-desktop.org.conf

213. By Terence Syrus <email address hidden>

merged changes from Daniel Hahler

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: