-
Notifications
You must be signed in to change notification settings - Fork 0
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
refactor: update to lastest version of gene-normalizer #146
Conversation
QueryHandler now accepts Database instance in the latest version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We'll need to update gene-norm dependencies to be 0.1.34+/0.2.9+ since that was a breaking change
@jsstevenson I'm not sure what the notation is for allowing multiple minor versions, while also excluding some older versions. I'm looking at this. |
@korikuzma yeah that's a good point. Pinning 0.1.29+ at least solves half of the problem (sort of joking, but sort of serious) |
@jsstevenson we could do something like this |
@korikuzma Fine w/ me |
* gene-normalizer versions allowed: >= 0.1.34 OR >= 0.2.9 * QueryHandler now accepts Database instance in the latest version * CoolSeqTool now has a gene_query_handler instance var
close #145
Needed for work in this variation normalizer issue