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

Remove default sasApp parameter and test without it #40

Closed
boomskats opened this issue Mar 12, 2017 · 1 comment
Closed

Remove default sasApp parameter and test without it #40

boomskats opened this issue Mar 12, 2017 · 1 comment
Assignees

Comments

@boomskats
Copy link
Contributor

https://github.com/Boemska/h54s/blob/master/dist/h54s.js#L53

This is causing (benign) JVM warnings on any version that is not 9.3. We should remove it from the default config and test

@bojan88
Copy link
Collaborator

bojan88 commented Apr 25, 2017

Removed in c55d45d, will be in 1.0 version

@bojan88 bojan88 closed this as completed Apr 25, 2017
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

2 participants