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

Adds Additional Undefined Check #104

Merged
merged 1 commit into from
Sep 30, 2024
Merged

Adds Additional Undefined Check #104

merged 1 commit into from
Sep 30, 2024

Conversation

rossgrambo
Copy link
Member

@rossgrambo rossgrambo commented Sep 27, 2024

typeof works great to see if a variable was declared. However the ? operator does not work when the variable was not declared.

For example:

> bad?.foo
Uncaught ReferenceError: bad is not defined
> bad = undefined
undefined
> bad?.foo
undefined

vs

> typeof bad
'undefined'
> bad = undefined
undefined
> typeof bad
'undefined'

@rossgrambo rossgrambo force-pushed the rossgrambo/process-fix branch from 0f43a87 to a24fd16 Compare September 27, 2024 19:44
@rossgrambo rossgrambo merged commit 5135f8e into main Sep 30, 2024
4 checks passed
@rossgrambo rossgrambo deleted the rossgrambo/process-fix branch September 30, 2024 17:56
linglingye001 pushed a commit that referenced this pull request Oct 17, 2024
linglingye001 pushed a commit that referenced this pull request Oct 24, 2024
zhiyuanliang-ms added a commit that referenced this pull request Oct 31, 2024
* Adds additional undefined check (#104)

* add loadCdn

* revert change

* fix lint

* update to loadFromCdn & reuse load method

* update

* update

* add js file extension to imports (#109)

* update typescript version (#112)

* add requestTracingOptions (#114)

* Version bump 1.1.1 (#115)

* bump up version 1.1.1

* update

---------

Co-authored-by: Ross Grambo <[email protected]>
Co-authored-by: linglingye001 <[email protected]>
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