Skip to content
This repository was archived by the owner on Feb 10, 2025. It is now read-only.

Conversation

@munificent
Copy link
Contributor

While I was at it, I:

  • Removed some unused imports.
  • Fixed a deprecated API call.
  • Removed the no-longer needed analysis options file.

While I was at it, I:

- Removed some unused imports.
- Fixed a deprecated API call.
- Removed the no-longer needed analysis options file.
@munificent munificent requested a review from nex3 July 12, 2018 21:35
@munificent
Copy link
Contributor Author

This is failing Travis on stable, but that's to be expected. I could remove stable from the Travis config, but I don't want to have to worry about re-adding that later after Dart 2 ships to stable. Is it OK if we just don't worry about that for now?

@kevmoo
Copy link
Contributor

kevmoo commented Jul 12, 2018

@munificent – just delete the travis config for stable...

Copy link
Contributor

@kevmoo kevmoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update changelog?

@munificent
Copy link
Contributor Author

just delete the travis config for stable...

Done.

Update changelog?

I, uh, already did in the original patch?

@kevmoo
Copy link
Contributor

kevmoo commented Jul 12, 2018

I, uh, already did in the original patch?

I are moron

@munificent munificent merged commit 75af367 into master Jul 13, 2018
@munificent munificent deleted the dart-2-ready branch July 13, 2018 00:16
mosuem pushed a commit to dart-lang/test that referenced this pull request Dec 12, 2024
…t-2-ready

Fix dart:convert "UTF8" -> "utf8".
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants