Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Bump psycopg requirement #5032

Merged
merged 1 commit into from
Apr 9, 2019
Merged

Bump psycopg requirement #5032

merged 1 commit into from
Apr 9, 2019

Conversation

richvdh
Copy link
Member

@richvdh richvdh commented Apr 9, 2019

We use execute_batch, which only arrived in psycopg 2.7.

@richvdh richvdh requested a review from a team April 9, 2019 10:07
@richvdh richvdh force-pushed the rav/bump_psycopg branch from 77ba432 to c013b55 Compare April 9, 2019 10:07
@erikjohnston
Copy link
Member

I guess this wasn't caught in CI because we test old deps only against sqlite...

@codecov
Copy link

codecov bot commented Apr 9, 2019

Codecov Report

Merging #5032 into develop will decrease coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #5032      +/-   ##
===========================================
- Coverage    61.59%   61.59%   -0.01%     
===========================================
  Files          332      332              
  Lines        34215    34215              
  Branches      5643     5643              
===========================================
- Hits         21076    21074       -2     
- Misses       11625    11628       +3     
+ Partials      1514     1513       -1

@richvdh richvdh merged commit 4abf5aa into develop Apr 9, 2019
@richvdh richvdh deleted the rav/bump_psycopg branch April 9, 2019 10:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants