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

address feedback from the PR #255 #268

Closed
lukasj opened this issue Apr 6, 2020 · 10 comments
Closed

address feedback from the PR #255 #268

lukasj opened this issue Apr 6, 2020 · 10 comments

Comments

@lukasj
Copy link
Contributor

lukasj commented Apr 6, 2020

major areas:

  • formatting
  • finish javax->jakarta transition
  • javadoc in code and spec should match
  • new schema in spec
@masterthefly
Copy link

made some minor changes to formatting in spec.adoc in my local and trying to verify "mvn verify -B -V -Poss-release -Dgpg.skip=true" ... got a build error
ERROR] Failed to execute goal org.apache.maven.plugins:maven-javadoc-plugin:3.1.1:jar (attach-javadocs) on project jakarta.persistence-api: MavenReportException: Error while generating Javadoc:
[ERROR] Exit code: 1 - javadoc: error - invalid flag: --frames
[ERROR]
[ERROR] Command line was: /usr/local/opt/openjdk/bin/javadoc @options @argfile

Anyone saw this before?

@lukasj
Copy link
Contributor Author

lukasj commented Apr 8, 2020

use jdk 11, not older, not newer for now...

I'll fix it in a while...

@lukasj
Copy link
Contributor Author

lukasj commented Apr 8, 2020

@viswanadhk it's going to be fixed by #270

@masterthefly
Copy link

thanks...openjdk11 worked but just fyi I see lot of warnings...
[WARNING] /Users/vishykasinadhuni/Documents/vishy-eclipse-foundation-development/jpa-api/api/src/main/java/jakarta/persistence/criteria/CriteriaBuilder.java:843: warning: no @param for
[WARNING] <E, C extends Collection> Predicate isMember(E elem, Expression collection);
[WARNING] ^

Thanks

@lukasj
Copy link
Contributor Author

lukasj commented Apr 8, 2020

#270 is in master. As for warnings - some of them are unfortunately not so easy to fix :-/

masterthefly added a commit to masterthefly/jpa-api that referenced this issue Apr 9, 2020
Signed-off-by: vishy kasinadhuni <viswanadhk@yahoo.com>
@jgrassel
Copy link
Member

Hello, looking at a build of the doc from a recent commit, I noticed

Copyright (c) 2019 Eclipse Foundation.

Should that be 2020?

@lukasj
Copy link
Contributor Author

lukasj commented Sep 16, 2020

@jgrassel https://github.com/eclipse-ee4j/jpa-api/blob/master/spec/src/main/asciidoc/license-efsl.adoc says ‘2019, 2020’. Are we looking st the same file?

@jgrassel
Copy link
Member

Presumably so. Here's a screen cap of the html doc (viewing) that I generated this morning from d9ddcb3:

image

@lukasj
Copy link
Contributor Author

lukasj commented Sep 21, 2020

@jgrassel that is not the final version of the spec document (your version probably has wrong definition of orm.xsd); use the head from main

@lukasj
Copy link
Contributor Author

lukasj commented Oct 26, 2020

I believe this is done

@lukasj lukasj closed this as completed Oct 26, 2020
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

3 participants