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: Make IS_MAYBE_MIRAGE simplified #9385

Merged
merged 1 commit into from
May 8, 2024

Conversation

MichalBryxi
Copy link
Contributor

Description

  • In previous version we fixed the code for FireFox that can now work in FireFox which has different .toString() implementation
  • This change should make it more error prone as it completely removes whitespace from the strings and then compares them
  • Follow-up for fix: Make IS_MAYBE_MIRAGE work in Firefox #9360

Notes for the release

@runspired runspired added 🎯 canary PR is targeting canary (default) 🏷️ chore This PR primarily refactors code or updates dependencies labels May 3, 2024
Copy link
Contributor

@runspired runspired left a comment

Choose a reason for hiding this comment

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

seems to fail both lint and ts parsing, probably a missed paren.

Thanks for the follow up :D

@MichalBryxi MichalBryxi force-pushed the mb/is-maybe-mirage-v2 branch from aa8f128 to 9fdde1b Compare May 6, 2024 05:31
@MichalBryxi
Copy link
Contributor Author

Sigh me :)

Maybe would help if I would not just ignore the hooks setup and figure out what is this error about:

git commit --amend
.git/hooks/pre-commit: line 2: ./node_modules/pre-commit/hook: No such file or directory

Ok, let's try one more time.

@MichalBryxi MichalBryxi force-pushed the mb/is-maybe-mirage-v2 branch from 9fdde1b to 7e921c6 Compare May 6, 2024 05:45
- In previous version we fixed the code for FireFox that can now work in FireFox which has different `.toString()` implementation
- This change should make it more error prone as it completely removes whitespace from the strings and then compares them
@MichalBryxi MichalBryxi force-pushed the mb/is-maybe-mirage-v2 branch from 7e921c6 to 81fcc8d Compare May 6, 2024 05:54
@runspired
Copy link
Contributor

@MichalBryxi we have no hooks! that means you cloned us at a point long ago when we did. You can safely remove that hook file/directory.

@runspired runspired merged commit 44655c6 into emberjs:main May 8, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎯 canary PR is targeting canary (default) 🏷️ chore This PR primarily refactors code or updates dependencies
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants