-
Notifications
You must be signed in to change notification settings - Fork 50
Major version increment. #104
Conversation
* Removes deprecated members, * Makes implementation classes private (they're not needed by end-users which just need a single instance). * Makes all the hash object constants. Tests run without change, so the removed functionality was *never tested*.
(As usual, a major version increment of a transitive dependency of |
pubspec.yaml
Outdated
typed_data: ^1.3.0-nullsafety | ||
|
||
dev_dependencies: | ||
pedantic: ^1.10.0-nullsafety | ||
test: ^1.16.0-nullsafety | ||
|
||
dependency_overrides: | ||
analyzer: |
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.
you should be able to get rid of all of these...right?
Or at least use pub overrides and not git overrides
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.
+1 on using pub overrides
pubspec.yaml
Outdated
typed_data: ^1.3.0-nullsafety | ||
|
||
dev_dependencies: | ||
pedantic: ^1.10.0-nullsafety | ||
test: ^1.16.0-nullsafety | ||
|
||
dependency_overrides: | ||
analyzer: |
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.
+1 on using pub overrides
Now uses pub overrides where available. Still needs an override for non-dev dependency |
* Removes deprecated members, * Makes implementation classes private (they're not needed by end-users which just need a single instance). * Makes all the hash object constants. Tests run without change, so the removed functionality was *never tested*.
Only override dev-dependencies. THIS DOES NOT WORK. The analyzer requires convert 2.0.0.
Still depend on convert override.
Co-authored-by: Nate Bosch <nbosch@google.com>
* Major version increment. * Removes deprecated members, * Makes implementation classes private (they're not needed by end-users which just need a single instance). * Makes all the hash object constants.
by end-users which just need a single instance).
Tests run without change, so the removed functionality was never tested.
WDYT?