8000 Rename getPermittedSubtypes() to permittedSubclasses() to fix build by KengoTODA · Pull Request #6 · forax/amber-record · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Rename getPermittedSubtypes() to permittedSubclasses() to fix build #6

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

KengoTODA
Copy link

Hi @forax,
thanks for your help at the core-libs-dev mailing list. It helped me to understand the record deeper. :)

When I compare your benchmark with mine, I found that the latest Travis build is broken:

src/main/java/fr.umlv.record/fr/umlv/sealed/NonSealedExample.java:14: error: cannot find symbol
		var permitted = I.class.getPermittedSubtypes();
		                       ^
  symbol:   method getPermittedSubtypes()
  location: variable class of type Class<I>

This is caused by an API change in Java 15; Java 15 uses Class#permittedSubclasses() instead of Class#getPermittedSubtypes().

note: I have no enough credit in Travis CI, so I haven't confirmed a successful build on Travis.
Instead, I confirmed it on GitHub Actions that uses this workflow definision.

Thanks again in advance!

@KengoTODA
Copy link
Author
KengoTODA commented Apr 24, 2021

The build is still failing, because the nodejs is too old (probably v8) and the latest npm packages won't work with it.
I'll try to specify nodejs version for Travis CI.

/home/travis/build/forax/amber-record/node_modules/decktape/decktape.js:251
  } catch {
          ^
SyntaxError: Unexpected token {

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.

1 participant
0