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

Feature template matrix #11

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

Conversation

gjroelofs
Copy link

Basic implementation of template and export into the generated matrix.
Maven plugin now uses Reflections to find required classes, and scans the entire classpath. (including dependencies)

Matrix will highlight rows of the Systems the template can participate in upon hover over Template.

@junkdog
Copy link
Owner

junkdog commented Mar 11, 2014

Hi, sorry for taking so long, this is way overdue - been submerged in my current game project, but I should manage to divide my time more evenly between projects now - a lot of stuff has happened in artemis-odb, like GWT support, so I'm really looking forward to releasing artemis/agrotera as soon as the feature issues have been impl.

I checked this PR into a local branch, building it now. Hopefully I can merge it before I must leave for work.

@junkdog
Copy link
Owner

junkdog commented Mar 11, 2014

The lombok fork would need to be published to sonatype, prerequisite for pushing releases - we could use net.onedaybeard, but how much work would it be to do a PR for lombok main? The repos appear to be pretty much in sync.

@junkdog
Copy link
Owner

junkdog commented Mar 11, 2014

Have to leave for work now: I couldn't get agrotera to compile - are you sure you included all commits in the PR? I noticed the agrotera poms were still referring to lombok-pg.

@gjroelofs
Copy link
Author

It happens; especially with pet projects; I've been swamped as well. I'll
try to take a look at it tonight.

On Tue, Mar 11, 2014 at 8:44 AM, Adrian Papari notifications@github.comwrote:

Have to leave for work now: I couldn't get agrotera to compile - are you
sure you included all commits in the PR? I noticed the agrotera poms were
still referring to lombok-pg.

Reply to this email directly or view it on GitHubhttps://github.com//pull/11#issuecomment-37270229
.

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.

None yet

2 participants