Skip to content

Added support for rhino scripting engine based PAC evaluation#62

Open
bergmannm wants to merge 1 commit intoMarkusBernhardt:masterfrom
bergmannm:master
Open

Added support for rhino scripting engine based PAC evaluation#62
bergmannm wants to merge 1 commit intoMarkusBernhardt:masterfrom
bergmannm:master

Conversation

@bergmannm
Copy link

Support for optional Rhino scripting engine PAC evaluation. Nashorn is kept as default.

@develar
Copy link
Contributor

develar commented Dec 10, 2018

#41 was merged but then Nashorn was dropped from JDK 11.

Also, we got a lot of requests that Nashorn is very slow (https://youtrack.jetbrains.com/issue/IDEA-202661, https://youtrack.jetbrains.com/issue/IDEA-203926).

So, this PR is wanted to be merged :)

@develar
Copy link
Contributor

develar commented Dec 12, 2018

As Nashorn is deprecated, I suggest simply revert #41 (I can prepare PR (JetBrains/intellij-deps-proxy-vole@3b9870c)). No need to add ability to use Nashorn as it is deprecated.

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