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

Switch to the new CodeCov uploader #13139

Merged
merged 1 commit into from
Oct 27, 2021

Conversation

MoonlightSentinel
Copy link
Contributor

@MoonlightSentinel MoonlightSentinel commented Oct 7, 2021

The currently used bash uploader is deprecated and will be removed in
the future[1]. This commit replaces the existing setup with the new
uploader for all currently supported platforms as proposed in [2].

Additional notes:

  • FreeBSD support isn't supported yet and keeps using the old uploader
  • Moved the new implementation into a dedicated file sourced by
    all coverage CI's (instead of changing each individual configuration)

[1] https://about.codecov.io/blog/codecov-uploader-deprecation-plan
[2] https://about.codecov.io/blog/introducing-codecovs-new-uploader

@dlang-bot
Copy link
Contributor

Thanks for your pull request and interest in making D better, @MoonlightSentinel! We are looking forward to reviewing it, and you should be hearing from a maintainer soon.
Please verify that your PR follows this checklist:

  • My PR is fully covered with tests (you can see the coverage diff by visiting the details link of the codecov check)
  • My PR is as minimal as possible (smaller, focused PRs are easier to review than big ones)
  • I have provided a detailed rationale explaining my changes
  • New or modified functions have Ddoc comments (with Params: and Returns:)

Please see CONTRIBUTING.md for more information.


If you have addressed all reviews or aren't sure how to proceed, don't hesitate to ping us with a simple comment.

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

Testing this PR locally

If you don't have a local development environment setup, you can use Digger to test this PR:

dub run digger -- build "master + dmd#13139"

@MoonlightSentinel MoonlightSentinel force-pushed the ci/migrate-codecov branch 5 times, most recently from d251e63 to af447bc Compare October 7, 2021 22:54
@maxhaton
Copy link
Member

maxhaton commented Oct 7, 2021

Azure also?

@MoonlightSentinel
Copy link
Contributor Author

Next one on the list after Cirrus

@MoonlightSentinel MoonlightSentinel force-pushed the ci/migrate-codecov branch 3 times, most recently from 6518a58 to 9b32d0b Compare October 8, 2021 00:22
Copy link
Member

@ljmf00 ljmf00 left a comment

Choose a reason for hiding this comment

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

We should think of adding https://pre-commit.com/ and shellcheck to the CI checks to check these trivial changes.

.azure-pipelines/windows.sh Outdated Show resolved Hide resolved
.azure-pipelines/windows.sh Outdated Show resolved Hide resolved
.azure-pipelines/windows.sh Outdated Show resolved Hide resolved
.azure-pipelines/windows.sh Outdated Show resolved Hide resolved
ci.sh Outdated Show resolved Hide resolved
The currently used bash uploader is deprecated and will be removed in
the future[1]. This commit replaces the existing setup with the new
uploader for all currently supported platforms as proposed in [2].

Additional notes:
- FreeBSD support isn't supported yet and keeps using the old uploader
- Moved the new implementation into a dedicated file `source`d by
  all coverage CI's (instead of changing each individual configuration)

[1] https://about.codecov.io/blog/codecov-uploader-deprecation-plan
[2] https://about.codecov.io/blog/introducing-codecovs-new-uploader
@MoonlightSentinel MoonlightSentinel marked this pull request as ready for review October 25, 2021 23:59
@MoonlightSentinel MoonlightSentinel changed the title Switch to new CodeCov uploader Switch to the new CodeCov uploader Oct 26, 2021
Copy link
Member

@ljmf00 ljmf00 left a comment

Choose a reason for hiding this comment

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

LGTM

@dlang-bot dlang-bot merged commit 9ff5805 into dlang:master Oct 27, 2021
MoonlightSentinel added a commit to MoonlightSentinel/druntime that referenced this pull request Oct 27, 2021
Reuse the script introduced in dlang/dmd#13139
MoonlightSentinel added a commit to MoonlightSentinel/phobos that referenced this pull request Oct 27, 2021
Reuse the script introduced in dlang/dmd#13139
dlang-bot pushed a commit to dlang/druntime that referenced this pull request Oct 27, 2021
dlang-bot pushed a commit to dlang/phobos that referenced this pull request Oct 27, 2021
@MoonlightSentinel MoonlightSentinel deleted the ci/migrate-codecov branch November 16, 2021 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants