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.

Sign up for GitHub

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

Jump to bottom

update some judgement condition #1225

Closed
wants to merge 1 commit into from
Closed

update some judgement condition #1225

wants to merge 1 commit into from

Conversation

mek1986
Copy link
Contributor

@mek1986 mek1986 commented Mar 18, 2023

1.== to ===
2.!= to !==

@mek1986
update some judgement condition
5b93eb9 
1.== to ===
2.!= to !==
@zloirock
Copy link
Owner

I thought about disallowing == / != for a long time, however, in the current core-js style it's allowed in safe cases. Any argumentation that in those cases should be used strict comparison?

@mek1986
Copy link
Contributor Author

mek1986 commented Mar 23, 2023

I thought about disallowing == / != for a long time, however, in the current core-js style it's allowed in safe cases. Any argumentation that in those cases should be used strict comparison?

Habit and style bar, no arguments。

@zloirock zloirock closed this in bc87aae Aug 1, 2023
@zloirock
Copy link
Owner

zloirock commented Aug 1, 2023

I enabled eqeqeq.

@mek1986
Copy link
Contributor Author

mek1986 commented Aug 1, 2023

I enabled eqeqeq.
06667ADD

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mek1986 @zloirock