Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

upgrade postgres.py from 2.1.2 to 2.2.0 #3781

Merged
merged 1 commit into from
Oct 5, 2015
Merged

Conversation

chadwhitacre
Copy link
Contributor

No description provided.

@techtonik
Copy link
Contributor

So, where is the postgres.py changelog to know what to test here?

@techtonik
Copy link
Contributor

Also it looks like downgrade in issue title.

@chadwhitacre chadwhitacre changed the title upgrade postgres.py from 2.1.2 to 2.0.0 upgrade postgres.py from 2.1.2 to 2.2.0 Sep 13, 2015
@chadwhitacre
Copy link
Contributor Author

Also it looks like downgrade in issue title.

Sorry, fixed. :-)

So, where is the postgres.py changelog to know what to test here?

I almost wrote one for you, but didn't. Sorry. :-(

You'll have to refer to the commit log for now.

@chadwhitacre
Copy link
Contributor Author

@techtonik Here are the two significant changes between 2.1.2 and 2.2.0:

liberapay/postgres.py#26
liberapay/postgres.py@55c69c0

@rohitpaulk
Copy link
Contributor

No issues raised, and this is marked as Review - going to merge.

rohitpaulk added a commit that referenced this pull request Oct 5, 2015
upgrade postgres.py from 2.1.2 to 2.2.0
@rohitpaulk rohitpaulk merged commit e2691ac into master Oct 5, 2015
@chadwhitacre chadwhitacre deleted the upgrade-postgres-py branch October 5, 2015 18:20
@chadwhitacre
Copy link
Contributor Author

Thanks @rohitpaulk! :-)

@rohitpaulk
Copy link
Contributor

Oops! Found a bug, opening a PR..

@rohitpaulk
Copy link
Contributor

liberapay/postgres.py#53

@rohitpaulk
Copy link
Contributor

cc: @whit537 - ^^ Production is going to break without that.

@chadwhitacre
Copy link
Contributor Author

Roger that.

@chadwhitacre
Copy link
Contributor Author

Blech. I mistook liberapay/postgres.py#53 for a gratipay.com PR. :-(

#3807 (comment)

@chadwhitacre
Copy link
Contributor Author

Rolled back ...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants