-
-
Notifications
You must be signed in to change notification settings - Fork 152
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 dry-types #563
Update dry-types #563
Conversation
21c75b0
to
9b6dedb
Compare
9b6dedb
to
576452e
Compare
32b7c0a
to
211ec49
Compare
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.
@@ -24,7 +24,7 @@ Gem::Specification.new do |spec| | |||
spec.add_runtime_dependency "rom", "~> 5.1" | |||
spec.add_runtime_dependency "rom-repository", "~> 5.1" | |||
spec.add_runtime_dependency "rom-sql", "~> 3.0" | |||
spec.add_runtime_dependency "dry-types", "~> 1.1" | |||
spec.add_runtime_dependency "dry-types", "~> 1.2" |
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.
WDYT if we set 1.3
version instead 1.2
? in this case, we'll able to use latest dry-system and I'll have an ability to test my library for dry-system dependency graph 🤔
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.
IIRC it wasn't available at the time I made the changes. I'll give it a try and see how it goes.
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.
Okay, updating to 1.3 introduces new issues. I'll merge this, and open a new one to 1.3 otherwise it will get confusing.
This makes model alpha releasable. =)
We had an issue with dry-types
1.2
removing.meta
from structs, which Nikita promptly fixed.