Document querydsl-maven-plugin #388

Closed
matthewadams opened this Issue Apr 8, 2013 · 7 comments

Comments

Projects
None yet
3 participants
@matthewadams
Contributor

matthewadams commented Apr 8, 2013

I couldn't find any docs on querydsl-maven-plugin. Had to google & refer to source, which was slower than it would've been with docs.

I think it's also worth documenting somewhere prominent, like in the apt-maven-plugin docs, that if you're using AspectJ to introduce artifacts into classes (id/version fields, audit fields, etc), then you need to do binary-based QueryDSL code generation, most likely at the process-classes phase at the earliest. Source code-based APT processing won't work due to the fact that ITDs aren't found by QueryDSL APT processors.

@timowest

This comment has been minimized.

Show comment
Hide comment
@timowest

timowest Apr 9, 2013

Member

It's documented here, but maybe it's a little bit isolated http://www.querydsl.com/static/querydsl/3.1.0/reference/html/ch03s03.html#d0e1681

Member

timowest commented Apr 9, 2013

It's documented here, but maybe it's a little bit isolated http://www.querydsl.com/static/querydsl/3.1.0/reference/html/ch03s03.html#d0e1681

@matthewadams

This comment has been minimized.

Show comment
Hide comment
@matthewadams

matthewadams Apr 9, 2013

Contributor

OMG I missed it. Jeez. Perhaps you could add standard Maven plugin-doc for both plugins (apt-maven-plugin & querydsl-maven-plugin), similar to maven-compiler-plugin or maven-install-plugin.

Contributor

matthewadams commented Apr 9, 2013

OMG I missed it. Jeez. Perhaps you could add standard Maven plugin-doc for both plugins (apt-maven-plugin & querydsl-maven-plugin), similar to maven-compiler-plugin or maven-install-plugin.

@timowest

This comment has been minimized.

Show comment
Hide comment
@timowest

timowest Apr 9, 2013

Member

The plugin doc is a good idea.

Member

timowest commented Apr 9, 2013

The plugin doc is a good idea.

@timowest

This comment has been minimized.

Show comment
Hide comment
@timowest

timowest Apr 21, 2013

Member

The plugin is now documented in the reference documentation 91f5ea6

I might add plugin docs later.

Member

timowest commented Apr 21, 2013

The plugin is now documented in the reference documentation 91f5ea6

I might add plugin docs later.

@timowest

This comment has been minimized.

Show comment
Hide comment
@timowest

timowest May 27, 2013

Member

Released in 3.2.0

Member

timowest commented May 27, 2013

Released in 3.2.0

@timowest timowest closed this May 27, 2013

@wdaogang

This comment has been minimized.

Show comment
Hide comment
@wdaogang

wdaogang Aug 22, 2013

how do I generate Q class with the @QueryProjection annotation via GenericExporter.

how do I generate Q class with the @QueryProjection annotation via GenericExporter.

@timowest

This comment has been minimized.

Show comment
Hide comment
@timowest

timowest Aug 22, 2013

Member

@wdaogang This is currently not supported.

Member

timowest commented Aug 22, 2013

@wdaogang This is currently not supported.

@timowest timowest added this to the 3.2.0 milestone Apr 14, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment