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
Asm #1054
Asm #1054
Conversation
Just a few comments: |
40dc149
to
3e2840b
Compare
Signed-off-by: Lukas Jungmann <lukas.jungmann@oracle.com>
Signed-off-by: Lukas Jungmann <lukas.jungmann@oracle.com>
Signed-off-by: Lukas Jungmann <lukas.jungmann@oracle.com>
add osgi headers to source bundle Signed-off-by: Lukas Jungmann <lukas.jungmann@oracle.com>
Signed-off-by: Lukas Jungmann <lukas.jungmann@oracle.com>
Signed-off-by: Lukas Jungmann <lukas.jungmann@oracle.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost OK except mentioned minor comments.
Two things which should be optionally included there:
- New Travis-CI target something like
...
matrix:
- MAVEN_TEST=asm
...
- if [ "$MAVEN_TEST" == "asm" ]; then
cd plugins/org.eclipse.persistence.asm/;
mvn test;
fi
- Maybe make generated Javadoc little bit nicer (title, copyright year)
.../src/main/java/org/eclipse/persistence/internal/libraries/asm/EclipseLinkASMClassWriter.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Lukas Jungmann <lukas.jungmann@oracle.com>
wrt travis for asm - it is non-trivial to add it correctly - the one built by travis won't be the one used by main eclispelink build there without extra modification of eclipselink's pom - I don't think we want to be doing that wrt javadoc - I can see correct cp year in the javadoc footer; I don't think we need to customize the default more |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
relates to #1051 in master: