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
chore: make macOS spellchecker fns formal no-ops #35514
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe a unittest to validate the noop on macOS ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 on the idea and the ifdef wrapping. Had a few minor suggestions.
85e7191
to
c41dfd9
Compare
Release Notes Persisted
|
I was unable to backport this PR to "19-x-y" cleanly; |
I was unable to backport this PR to "20-x-y" cleanly; |
I have automatically backported this PR to "21-x-y", please check out #35679 |
* chore: make macOS spellchecker fns formal no-ops * docs: correct no-op note * test: add no-op specs
Description of Change
Closes #35508.
Formally make
session.setSpellCheckerLanguages()
andsession.setSpellCheckerDictionaryDownloadURL()
no-ops on macOS. macOS uses system spellchecking functionality so those functions would not have their intended effect, but could still throw errors if called with an unsupported language.Checklist
npm test
passesRelease Notes
Notes: Fixed an issue where some spellcheck functionality would incorrectly throw errors on macOS.