lycantropos / hypothesis_sqlalchemy

hypothesis strategies for generating SQLAlchemy objects
MIT License
28 stars 8 forks source link

Relax upper bound on `sqlalchemy`, to allow 2.0 #39

Closed dycw closed 1 year ago

dycw commented 1 year ago

Resolves #38.

lycantropos commented 1 year ago

@dycw thanks, it looks like we need to merge #40 first since Python3.6 is no longer supported by Github Actions (and in general)

and also it will require bumping minor version because we are going to add Python3.11 & SQLAlchemy>=2 support (which doesn't sound like a bugfix)

codecov[bot] commented 1 year ago

Codecov Report

Merging #39 (dfce869) into master (ea5a4db) will not change coverage. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #39   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           13        13           
  Lines          224       230    +6     
=========================================
+ Hits           224       230    +6     
Impacted Files Coverage Δ
hypothesis_sqlalchemy/__init__.py 100.00% <100.00%> (ø)
hypothesis_sqlalchemy/core/column.py 100.00% <100.00%> (ø)
hypothesis_sqlalchemy/core/column_type.py 100.00% <100.00%> (ø)
hypothesis_sqlalchemy/core/columns_records.py 100.00% <100.00%> (ø)
hypothesis_sqlalchemy/core/enum.py 100.00% <100.00%> (ø)
hypothesis_sqlalchemy/core/hints.py 100.00% <100.00%> (ø)
hypothesis_sqlalchemy/core/table.py 100.00% <100.00%> (ø)
hypothesis_sqlalchemy/core/table_constraints.py 100.00% <100.00%> (ø)
hypothesis_sqlalchemy/core/table_records.py 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

lycantropos commented 1 year ago

@dycw : I've already bumped version on your branch (sorry if it disturbs you), also added mypy support, let's merge once checks pass

dycw commented 1 year ago

Hi @lycantropos , sorry I think we're writing at the same time, and I've force pushed some stuff.

lycantropos commented 1 year ago

this should do, let's wait for checks to pass and merge it, after that we will release v1.1.0

dycw commented 1 year ago

Thanks for this great library again!

lycantropos commented 1 year ago

@dycw: you are welcome, thanks for your contribution