-
Notifications
You must be signed in to change notification settings - Fork 35
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
Migrated jnosql-arangodb-extension
into the jnosql-arangodb
module;
#212
Conversation
Signed-off-by: Maximillian Arruda <dearrudam@gmail.com>
Signed-off-by: Maximillian Arruda <dearrudam@gmail.com>
…clipse.jnosql.databases Signed-off-by: Maximillian Arruda <dearrudam@gmail.com>
Signed-off-by: Maximillian Arruda <dearrudam@gmail.com>
Signed-off-by: Maximillian Arruda <dearrudam@gmail.com>
Signed-off-by: Maximillian Arruda <dearrudam@gmail.com>
Signed-off-by: Maximillian Arruda <dearrudam@gmail.com>
@otaviojava could you review this PR, please? |
Can one of the admins verify this patch? |
pom.xml
Outdated
@@ -103,6 +104,18 @@ | |||
<artifactId>jnosql-mapping-key-value</artifactId> | |||
<version>${project.version}</version> | |||
</dependency> | |||
<dependency> |
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.
I've updated the pom parent; we don't need this anymore.
Could you check it, please?
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.
@otaviojava, it's done!
@dearrudam please, update your branch to take the newest code from the main branch.
|
About the package, please, use:
|
Signed-off-by: Maximillian Arruda <dearrudam@gmail.com>
Signed-off-by: Maximillian Arruda <dearrudam@gmail.com>
Signed-off-by: Maximillian Arruda <dearrudam@gmail.com>
Signed-off-by: Maximillian Arruda <dearrudam@gmail.com>
Signed-off-by: Maximillian Arruda <dearrudam@gmail.com>
…ures Signed-off-by: Maximillian Arruda <dearrudam@gmail.com>
@otaviojava, please, review this one?! |
…ures Signed-off-by: Maximillian Arruda <dearrudam@gmail.com>
import java.lang.annotation.Target; | ||
|
||
/** | ||
* To a dynamic query on ArangoDBRepository and ArangoDBRepositoryAsync interfaces. |
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.
There is no ArangoDBRepositoryAsync
could you update this one on another PR?
@Target(ElementType.METHOD) | ||
public @interface AQL { | ||
|
||
String value(); |
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.
on another PR, could you document this annotation?
Refs: eclipse/jnosql#355
Changes
jnosql-arangodb-extension
into thejnosql-arangodb
module;