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

Update ReadMe in section Example for 3.1.x. #33

Open
wants to merge 1 commit into
base: release-3.1
Choose a base branch
from
Open

Update ReadMe in section Example for 3.1.x. #33

wants to merge 1 commit into from

Conversation

Naoghuman
Copy link

Project still under development?

Project still under development?
@rmuller
Copy link
Owner

rmuller commented Feb 15, 2016

Thanks for the contribution.

I do not understand the changes you made. You changed the method references to lambdas? What is the idea behind this? I prefer the method references because they are more concise and equally clear.

No, this code is not under development any more. The 3.1 version is in production since 2014. However the 3.0 code has some contributions i want to add to the 3.1 code base.

@Naoghuman
Copy link
Author

Hi,
thx for your answer.

I haven't see any release 3.1.x in the release section (newest is infomas-asl-3.0.5 from 19.Oct.2015). I want to integrate you branch release-3.1 (or the release jar 3.1) directly in my little library Lib-Action ( https://github.com/Naoghuman/lib-action) because I want to use the Lambda version (3.1.x) there.

The original example with method references in section Example Usage (3.1.x versions) in the ReadMe don't work for me. When I changed to the lamdas (see in my commit) then the code works and I can scan the classes and methods like in your example with my own annotations. See also the method scan() in ActionFacade#scan().

So I think there is a typo in your example section.

@rmuller
Copy link
Owner

rmuller commented Feb 15, 2016

Ok, clear. Will take a look at it. Will be next Wednesday.

And the code is not released to Maven Central. Will fix that too (this todo is way too long overdue :( )

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.

2 participants