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

isBrowserEnvironment is false in the browser #459

Open
shulandmimi opened this issue Aug 28, 2022 · 6 comments
Open

isBrowserEnvironment is false in the browser #459

shulandmimi opened this issue Aug 28, 2022 · 6 comments

Comments

@shulandmimi
Copy link

running in a strict mode in the browser will lead isBrowserEnvironment is false

"use strict";

var isBrowserEnvironment = (function() {
    // eslint-disable-next-line no-undef
    return (typeof window !== "undefined") && (this === window);
}).call();

isBrowserEnvironment
// false
@ochameau
Copy link
Contributor

Is this on an old branch? This variable doesn't exist.

@ippeiukai
Copy link

ippeiukai commented Oct 19, 2022

It is fixed in the master branch indeed.
#363

Still present in the latest version.
https://github.com/mozilla/source-map/blob/v0.7.4/lib/read-wasm.js#L4

@shulandmimi
Copy link
Author

Is this on an old branch? This variable doesn't exist.

https://www.npmjs.com/package/source-map/v/0.7.4 still exist

Is there a plan to release the next version?

@ochameau
Copy link
Contributor

Yes, we are planning to release a new stable version.

No ETA yet. We are trying to revive the maintenance of this repo which has been stale for a while.

@shulandmimi
Copy link
Author

thanks

@eliasm307
Copy link

Hello, is there any update regarding this? it has been 2 years since the last comment and the patched version has not been released?

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

No branches or pull requests

4 participants