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(magic-value): make incrementing magic value a manual step #641

Merged
merged 1 commit into from Oct 16, 2021
Merged

Conversation

bcoe
Copy link
Member

@bcoe bcoe commented Oct 16, 2021

We should not change the hash used to identify coverage reports unless
an incompatible change is actually introduced. In practice, we should
try to make sure changes are backwards compatible.

Fixes #631

We should not change the hash used to identify coverage reports unless
an incompatible change is actually introduced. In practice, we should
try to make sure changes are backwards compatible.
@bcoe bcoe merged commit 823010b into master Oct 16, 2021
@bcoe bcoe deleted the no-inc branch October 16, 2021 20:34
@bcoe bcoe restored the no-inc branch October 16, 2021 20:34
@bcoe bcoe deleted the no-inc branch October 16, 2021 20:34
vivek-freshworks pushed a commit to freshworks/istanbuljs that referenced this pull request Oct 16, 2023
…uljs#641)

We should not change the hash used to identify coverage reports unless
an incompatible change is actually introduced. In practice, we should
try to make sure changes are backwards compatible.
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.

istanbul-lib-instrument@5's readInitialCoverage is broken
1 participant