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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release chai@4.3.0 #1266

Closed
wants to merge 1 commit into from
Closed

Release chai@4.3.0 #1266

wants to merge 1 commit into from

Conversation

lucasfcosta
Copy link
Member

@lucasfcosta lucasfcosta commented Jun 30, 2019

Summary

Hello everyone 馃槉

This is a release for 4.3.0. It doesn't contain that many changes but I think it's important to release this one ASAP so that we can get this feature out.

#1257 is important for jest to be able to show better diffs for people using chai as their assertion library.

As per instructions in CONTRIBUTING.md a draft release is available at https://github.com/chaijs/chai/releases/tag/untagged-86c01ce248840cb410d6 and will be published once we get this version out. It's been a while since I've last released a chai version so please let me know if I missed anything.

The next step is to get this in and publish the release so that the package can be published to NPM, correct?

I've seen this post though and it seems like we might have to manually publish.

PS: Travis seems to be failing because SauceLabs is under maintenance for free users.

@lucasfcosta lucasfcosta requested a review from a team as a code owner June 30, 2019 16:16
@lucasfcosta lucasfcosta requested a review from a team June 30, 2019 16:18
@codecov
Copy link

codecov bot commented Jun 30, 2019

Codecov Report

Merging #1266 into master will increase coverage by 0.87%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1266      +/-   ##
==========================================
+ Coverage   94.61%   95.48%   +0.87%     
==========================================
  Files          33       33              
  Lines        1708     2082     +374     
  Branches      416      583     +167     
==========================================
+ Hits         1616     1988     +372     
- Misses         92       94       +2
Impacted Files Coverage 螖
lib/chai.js 100% <100%> (酶) 猬嗭笍
lib/chai/core/assertions.js 99.42% <0%> (+0.02%) 猬嗭笍

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 03913cb...8ae00a0. Read the comment docs.

Copy link
Member

@keithamus keithamus left a comment

Choose a reason for hiding this comment

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

LGTM 馃檪

Happy to make the release manually if travis fails

Copy link
Member

@vieiralucas vieiralucas left a comment

Choose a reason for hiding this comment

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

Let鈥檚 do this 馃殌

@keithamus
Copy link
Member

We're getting test failures on this branch. I won't have time to look at them any time soon, but if one of the other maintainers could, then I can cut a release.

@keithamus keithamus mentioned this pull request Nov 8, 2019
@lucasfcosta
Copy link
Member Author

I'm taking a look at this after my Holidays. I'm back next week, so should be good to go by the weekend.

@Stanzilla
Copy link

Is this still happening?

@keithamus keithamus closed this Feb 4, 2021
@keithamus keithamus deleted the 4.3.0 branch March 2, 2021 18:22
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.

None yet

4 participants