Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update gtfparse version requirement #118

Merged
merged 3 commits into from
Dec 28, 2023
Merged

Conversation

jpintar
Copy link
Contributor

@jpintar jpintar commented Dec 23, 2023

In 00ece88, gtfparse version was pinned to <2 because of confilcts between gtfparse and recent versions of polars. Since 12/21/23, with gtfparse v. 2.1.0, the conflicts have been eliminated, so there is no need to keep older versions pinned.

In 00ece88, gtfparse version was pinned to <2 because of conflicts between gtfparse and recent versions of polars. Since 12/21/23, with gtfparse v. 2.1.0, the conflicts have been eliminated, so there is no need to keep older versions pinned.
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (12c103f) 56.20% compared to head (0b8d08d) 56.20%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #118   +/-   ##
=======================================
  Coverage   56.20%   56.20%           
=======================================
  Files          13       13           
  Lines         427      427           
=======================================
  Hits          240      240           
  Misses        187      187           

@grst
Copy link
Member

grst commented Dec 28, 2023

Thanks for the heads-up!

@grst grst enabled auto-merge (squash) December 28, 2023 20:53
@grst grst merged commit 527284c into icbi-lab:main Dec 28, 2023
6 checks passed
@jpintar jpintar deleted the gtfparse-version-fix branch December 29, 2023 01:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants