Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ci: remove coveralls #92

Merged
merged 3 commits into from
Aug 5, 2020
Merged

ci: remove coveralls #92

merged 3 commits into from
Aug 5, 2020

Conversation

QmarkC
Copy link
Contributor

@QmarkC QmarkC commented Aug 4, 2020

This removes coveralls for the coverage NPM script. In the switch to GitHub actions as part of #89 the coverage script was broken with error Couldn't find a repository matching this job as a result of switching from travis CI.

See the renovate PR #82 as an example, https://github.com/yargs/y18n/pull/82/checks?check_run_id=943953063. This PR should resolve that issue.

@QmarkC
Copy link
Contributor Author

QmarkC commented Aug 5, 2020

Updated dependencies nyc (#81) and rimraf (#82) to latest versions to resolve issues on Windows. Also tested with local Windows 10 PC.

@QmarkC QmarkC requested a review from bcoe August 5, 2020 01:25
@bcoe bcoe merged commit 2360586 into yargs:master Aug 5, 2020
@bcoe
Copy link
Member

bcoe commented Aug 5, 2020

thanks @QmarkC 👌

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

Successfully merging this pull request may close these issues.

2 participants