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

More chai/sinon patches #315

Closed
wants to merge 36 commits into from
Closed

More chai/sinon patches #315

wants to merge 36 commits into from

Conversation

danbeam
Copy link
Contributor

@danbeam danbeam commented Jun 29, 2022

Alright, let me explain what's going on here:

@danbeam danbeam marked this pull request as draft June 29, 2022 22:04
@codecov
Copy link

codecov bot commented Jun 29, 2022

Codecov Report

Merging #315 (e37e886) into main (93d99e0) will decrease coverage by 0.08%.
The diff coverage is 86.95%.

@@            Coverage Diff             @@
##             main     #315      +/-   ##
==========================================
- Coverage   92.18%   92.09%   -0.09%     
==========================================
  Files          26       26              
  Lines        1842     1872      +30     
  Branches      381      392      +11     
==========================================
+ Hits         1698     1724      +26     
  Misses        102      102              
- Partials       42       46       +4     
Impacted Files Coverage Δ
src/cli/index.ts 0.00% <0.00%> (ø)
src/utils/consts.ts 100.00% <ø> (ø)
src/transformers/chai-should.ts 94.78% <75.00%> (-0.80%) ⬇️
src/transformers/sinon.ts 87.50% <96.66%> (+0.83%) ⬆️
src/cli/transformers.ts 94.11% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 93d99e0...e37e886. Read the comment docs.

@danbeam danbeam marked this pull request as ready for review June 29, 2022 22:11
danbeam and others added 27 commits June 29, 2022 15:16
* Fail build if CLI does not respond

* Downgrade globby to fix ESM issue

* Revert "Correct globby import"

This reverts commit a0cb8eb.

* Bump version

* Downgrade execa

* Revert "Correct execa import"

This reverts commit 429c438.
@danbeam danbeam changed the title More patches More chai/sinon patches Jun 29, 2022
@skovhus
Copy link
Owner

skovhus commented Jun 30, 2022

So you are trying to rewrite the git history here to fix the wrong emails in the commits? Normally one would never rewrite history for the main branch. Let me know why this is important to you. Also feel free to DM (e.g. on twitter).

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.

4 participants