Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ARQ-438 @DeploymentName and @OperatesOnDeployment could be made typesafe #16

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

DavideD
Copy link
Member

@DavideD DavideD commented Aug 28, 2011

I've created this quick prototype.

What do you think?

Cheers

@aslakknutsen
Copy link
Member

holding this of for 1.1

@kpiwko
Copy link
Member

kpiwko commented Sep 12, 2011

I though that idea could be taken one step further, e.g. no String value required in the annotation.

String deploymentName(Method deploymentMethod) method can for example return the fully-qualified name of the annotation or it's hashcode.

@DavideD
Copy link
Member Author

DavideD commented Sep 12, 2011

I see.

I'll try to create a version that works that way

@DavideD
Copy link
Member Author

DavideD commented Sep 19, 2011

Now it should work as suggested.

@DavideD
Copy link
Member Author

DavideD commented Sep 20, 2011

It uses the class name of the annotation as name of the deployment.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants