-
Notifications
You must be signed in to change notification settings - Fork 75
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
feat(isthumus): extract isthmus CLI into isthmus-cli project #248 #249
Conversation
2abc3e0
to
3e40c36
Compare
Reverted change to .editorconfig (despite helping me to get correct formatting locally) as the change applied to all files and so fails indentation checks for some file types. |
Once #244 is merged, I will rebase on that change and mark this PR for review. |
This avoid transitive dependencies on graal-sdk and picocli components for projects that consume isthmus as a library.
3e40c36
to
98a2457
Compare
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.
Changes look good. Thanks for splitting this out.
|
||
dependencies { | ||
implementation(project(":core")) | ||
implementation("org.apache.calcite:calcite-core:${CALCITE_VERSION}") | ||
implementation("org.apache.calcite:calcite-server:${CALCITE_VERSION}") | ||
implementation("org.junit.jupiter:junit-jupiter:${JUNIT_VERSION}") | ||
testImplementation("org.junit.jupiter:junit-jupiter:${JUNIT_VERSION}") |
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.
good call updating this as well
This avoid transitive dependencies on graal-sdk and picocli components for projects that consume isthmus as a library.