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

fix: Correctly load nycrc #53

Merged
merged 1 commit into from
Dec 2, 2022
Merged

fix: Correctly load nycrc #53

merged 1 commit into from
Dec 2, 2022

Conversation

yusufkandemir
Copy link
Contributor

loadNycConfig is async:
https://github.com/istanbuljs/load-nyc-config/blob/2033a007672d90669c48c79e6a2d63a3cd0851a7/index.js#L139-L164

But, it's handled as if it's a synchronous one. So, nycrc isn't being loaded or respected correctly.

@IlCallo
Copy link

IlCallo commented Nov 29, 2022

Hi there, we're waiting for this fix to be released to release latest Quasar Cypress integration package version as stable
Any hope it can be merged and released soon?

@iFaxity
Copy link
Owner

iFaxity commented Dec 2, 2022

Hi, sorry for the long wait. I will review this PR today and approve it as soon as possible :).

@iFaxity
Copy link
Owner

iFaxity commented Dec 2, 2022

LGTM!
Merging and a new release should be out within a few minutes.

Thanks for the contribution @yusufkandemir.

@iFaxity iFaxity merged commit 5cc94f1 into iFaxity:next Dec 2, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Dec 2, 2022

🎉 This PR is included in version 3.0.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants