search

clamsproject / clams-python

CLAMS SDK for python
http://sdk.clams.ai/
Apache License 2.0
4 stars 1 forks source link

updating dependency versions #104

Closed keighrim closed 1 year ago

keighrim commented 1 year ago

This PR includes updates on

  1. minimum python requirements (also see https://github.com/clamsproject/mmif-python/issues/191)
  2. flask version (1 -> 2) (see #94)
  3. "uncapping" other dependency versions. (see #89)

Using a new major version of flask needs more testing, which I'm working on.

codecov[bot] commented 1 year ago

Codecov Report

Merging #104 (4c8af17) into develop (24d3cf4) will increase coverage by 0.95%. The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #104      +/-   ##
===========================================
+ Coverage    79.58%   80.53%   +0.95%     
===========================================
  Files            6        5       -1     
  Lines          431      411      -20     
===========================================
- Hits           343      331      -12     
+ Misses          88       80       -8
Flag Coverage Δ
unittests 80.53% <100.00%> (+0.95%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
clams/appmetadata/__init__.py 89.91% <100.00%> (+0.17%) :arrow_up:
clams/__init__.py
clams/app/__init__.py 86.00% <0.00%> (+0.14%) :arrow_up:
clams/source/__init__.py 69.38% <0.00%> (+0.70%) :arrow_up:
clams/restify/__init__.py 76.92% <0.00%> (+1.06%) :arrow_up:

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

keighrim commented 1 year ago

Manual tests with clams apps written for the latest clams-python release (0.5.1) did not show any problems with these dependency updates. I'm merging this.