pub_api_client icon indicating copy to clipboard operation
pub_api_client copied to clipboard

Use package:pubspec_parse

Open xvrh opened this issue 2 years ago • 4 comments

This is a proposal to replace package:pubspec by package:pubspec_parse. pubspec_parse is a package owned by the Dart team and seems to have a better quality score. Also it exposes more fields (like repository, issueTracker...)

xvrh avatar Jun 20 '22 15:06 xvrh

@xvrh that makes sense, I believe initially the other package was chosen due to less dependencies and we were having a lot of conflicts with yaml, collection and so on, but maybe these issues have been worked out.

leoafarias avatar Jun 20 '22 15:06 leoafarias

@leoafarias in terms of dependencies, this is also a win:

These packages are no longer being depended on:
- pubspec 2.2.0
- quiver 3.1.0
- uri 1.0.0

pubspec_parse was already pulled as a the transitive dependency.

xvrh avatar Jun 20 '22 15:06 xvrh

@xvrh can you fix the metrics test on this PR to make sure there is no issues with your previous PR?

leoafarias avatar Jun 20 '22 15:06 leoafarias

@leoafarias done, I rebased my PR on top of the latest main

xvrh avatar Jun 20 '22 15:06 xvrh