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: do not call equal with an uninitialized value #107

Merged

Conversation

divdavem
Copy link
Member

@divdavem divdavem commented Oct 3, 2023

No description provided.

@divdavem divdavem force-pushed the dontCallEqualWithUninitializedValue branch from 41c2cdf to 9ac0756 Compare October 4, 2023 08:36
@codecov
Copy link

codecov bot commented Oct 4, 2023

Codecov Report

Merging #107 (fa4b4d2) into master (4b8f678) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #107      +/-   ##
==========================================
+ Coverage   98.58%   98.62%   +0.03%     
==========================================
  Files           1        1              
  Lines         354      364      +10     
  Branches       63       64       +1     
==========================================
+ Hits          349      359      +10     
  Partials        5        5              
Files Coverage Δ
src/index.ts 98.62% <100.00%> (+0.03%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@divdavem divdavem marked this pull request as ready for review October 4, 2023 08:38
@divdavem divdavem force-pushed the dontCallEqualWithUninitializedValue branch from 9ac0756 to 663f75a Compare October 4, 2023 08:41
@divdavem divdavem force-pushed the dontCallEqualWithUninitializedValue branch from 663f75a to fa4b4d2 Compare October 4, 2023 08:43
@divdavem divdavem requested a review from fbasso October 4, 2023 08:46
Copy link
Contributor

@fbasso fbasso left a comment

Choose a reason for hiding this comment

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

LGTM !

@divdavem
Copy link
Member Author

divdavem commented Oct 4, 2023

@fbasso Thank you for your review!

@divdavem divdavem merged commit f6115d9 into AmadeusITGroup:master Oct 4, 2023
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.

2 participants