buildpg icon indicating copy to clipboard operation
buildpg copied to clipboard
verified publisher icon samuelcolvin

Add Having clause

Open euri10 opened this issue 2 years ago • 4 comments

Unfortunately it is missing now and I have to concatenate stuff :) Will probably look also for bool_or etc..but probably in another PR

euri10 avatar Apr 01 '23 14:04 euri10

Codecov Report

Merging #44 (84a5837) into master (e2a16ab) will not change coverage. The diff coverage is 100.00%.

Additional details and impacted files 
@@            Coverage Diff            @@
##            master       #44   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            8         8           
  Lines          554       556    +2     
  Branches        50        50           
=========================================
+ Hits           554       556    +2

codecov[bot] avatar Apr 01 '23 14:04 codecov[bot]

the lint failing is because of black pin, do you want me to upgrade it @samuelcolvin in this PR or would prefer another ?

euri10 avatar Apr 01 '23 15:04 euri10

happy to accept the change, but please can we update click or black or whatever to get tests passing?

samuelcolvin avatar Apr 02 '23 13:04 samuelcolvin

happy to accept the change, but please can we update click or black or whatever to get tests passing?

here it is

euri10 avatar Apr 04 '23 05:04 euri10