Skip to content
This repository was archived by the owner on Oct 17, 2024. It is now read-only.

Major version increment. #104

Merged
merged 11 commits into from
Dec 10, 2020
Merged

Major version increment. #104

merged 11 commits into from
Dec 10, 2020

Conversation

lrhn
Copy link
Contributor

@lrhn lrhn commented Nov 12, 2020

  • Is null safe.
  • 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.

WDYT?

* 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*.
@lrhn lrhn requested review from kevmoo and natebosch November 12, 2020 14:21
@google-cla google-cla bot added the cla: yes label Nov 12, 2020
@lrhn
Copy link
Contributor Author

lrhn commented Nov 12, 2020

(As usual, a major version increment of a transitive dependency of test is a problem. We need to update web_socket_channel to support crypto 3.0.0 before we can release crypto 3.0.0.

pubspec.yaml Outdated
typed_data: ^1.3.0-nullsafety

dev_dependencies:
pedantic: ^1.10.0-nullsafety
test: ^1.16.0-nullsafety

dependency_overrides:
analyzer:
Copy link
Contributor

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

Copy link
Contributor

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:
Copy link
Contributor

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

@lrhn
Copy link
Contributor Author

lrhn commented Nov 26, 2020

Now uses pub overrides where available.

Still needs an override for non-dev dependency convert.
We need to make analyzer allow convert >3.0.0 before that can be removed.

lrhn added 4 commits November 26, 2020 17:06
* 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.
@lrhn lrhn merged commit c89a5be into master Dec 10, 2020
@lrhn lrhn deleted the 3.0 branch December 10, 2020 17:00
mosuem pushed a commit to dart-lang/core that referenced this pull request Oct 15, 2024
* 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.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

4 participants