Skip to content

Pin sqlalchemy!=1.4.0#996

Merged
dwhswenson merged 2 commits into
openpathsampling:masterfrom
dwhswenson:temp-pin-sqlalchemy
Mar 17, 2021
Merged

Pin sqlalchemy!=1.4.0#996
dwhswenson merged 2 commits into
openpathsampling:masterfrom
dwhswenson:temp-pin-sqlalchemy

Conversation

@dwhswenson

Copy link
Copy Markdown
Member

Tests stopped working this morning (had to be the day of my APS March Meeting talk!) SQLAlchemy accidentally made changes in their 1.4.0 release that broke SimStore. See sqlalchemy/sqlalchemy#6074. Looks like they're already preparing a fix, so I'm pinning to !=1.4.0 as opposed to <1.4.

@codecov

codecov Bot commented Mar 17, 2021

Copy link
Copy Markdown

Codecov Report

Merging #996 (5a863e5) into master (d303ff6) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #996   +/-   ##
=======================================
  Coverage   80.58%   80.58%           
=======================================
  Files         138      138           
  Lines       14671    14671           
=======================================
  Hits        11823    11823           
  Misses       2848     2848           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d303ff6...5a863e5. Read the comment docs.

@sroet

sroet commented Mar 17, 2021

Copy link
Copy Markdown
Member

LGTM, good luck on the talk (assuming it hasn't happened already);)

@dwhswenson dwhswenson merged commit a127251 into openpathsampling:master Mar 17, 2021
@dwhswenson dwhswenson deleted the temp-pin-sqlalchemy branch March 17, 2021 21:27
@dwhswenson

Copy link
Copy Markdown
Member Author

Going on in a half hour -- almost forgot to merge this first! (It's a conference in the US, so I'm speaking at 11pm local.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants