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

this has been rewritten to undefined on the new version of rollup #162

Open
iceleo-com opened this issue Apr 1, 2021 · 3 comments
Open

Comments

@iceleo-com
Copy link

Hi sir/madam,

I run into this issue with the latest version of rollup js

(!) `this` has been rewritten to `undefined`
https://rollupjs.org/guide/en/#error-this-is-undefined
node_modules/detect-browser/es/index.js
1: var __spreadArrays = (this && this.__spreadArrays) || function () {
                         ^
2:     for (var s = 0, i = 0, il = arguments.length; i < il; i++) s += arguments[i].length;
3:     for (var r = Array(s), k = 0, i = 0; i < il; i++)
...and 1 other occurrence

May you adjust your lib to work with rollup js?

Thankyou for your hard work!

@DamonOehlman
Copy link
Owner

I will definitely take a look - thanks for reporting the issue!

@iulo
Copy link

iulo commented Feb 25, 2022

still got this warning. I read @xon52 refs, seems work when set this package in external. But I need make it bundle in my package. Any help is appreciated

@deeeed
Copy link

deeeed commented Jan 9, 2023

+1 on this issue

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