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.

Sign up for GitHub

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

Jump to bottom

Remove tests from package #118

Open
lsantos10 opened this issue Jul 27, 2018 · 2 comments · May be fixed by #136
Open

Remove tests from package #118

lsantos10 opened this issue Jul 27, 2018 · 2 comments · May be fixed by #136

Comments

@lsantos10
Copy link

Could the tests be removed from the package?
This makes the packages bigger and contain unnecessary information.
@thib3113
Copy link
Contributor

And some CVE detected with the jquery file ...

CVE-2015-9251 : jQuery before 3.0.0 is vulnerable to Cross-site Scripting (XSS) attacks when a cross-domain Ajax request is performed without the dataType option, causing text/javascript responses to be executed.

Please remove them !

@thib3113 thib3113 linked a pull request Oct 4, 2019 that will close this issue
Open
@dev6james
Copy link

Another one for moment.

#137

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

create ignore to remove test folder from the npm lib
3 participants
@thib3113 @dev6james @lsantos10