Skip to content
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

Dart2 compatibility #14

Closed
wants to merge 3 commits into from
Closed

Dart2 compatibility #14

wants to merge 3 commits into from

Conversation

ChacaS0
Copy link

@ChacaS0 ChacaS0 commented Mar 24, 2019

No description provided.

@joelbrostrom
Copy link

Will this be merged or is the repo dead?

@feliwir
Copy link

feliwir commented Nov 10, 2019

Can this be merged pleased indeed?

@hootener
Copy link

@ChacaS0 Happy to merge this, but as Dart isn't a core competency of Codecov's engineering staff, would it be possible for you to point to the following:

  1. A public repo on a publicly viewable CI page (e.g., Travis, CircleCI, Azure Pipelines, etc. demonstrating codecov/dart successfully uploading a coverage report to codecov
  2. The URL to that commit on codecov.io
  3. Comment on this PR @'ing me with URLs pointing to 1 and 2.

Sorry to require the extra steps in this case, but it's likely the most straightforward way to demonstrate that this PR will bring Dart2 compatibility since I don't have the immediate tools or expertise to get this verified in-house.

Anyone else interested in seeing this PR merged is more than welcome to perform the above. I'll merge the PR as soon as correctness of implementation can be shown.

@thomasrockhu thomasrockhu reopened this Sep 9, 2020
@thomasrockhu
Copy link
Contributor

Closing out in favor of #14. Any help there would be greatly appreciated @ChacaS0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants