-
Notifications
You must be signed in to change notification settings - Fork 363
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
[SW-1304] Definition of assembly jar via transitive exclusions #1221
Conversation
@@ -52,6 +30,8 @@ shadowJar { | |||
relocate 'org.eclipse.jetty', 'ai.h2o.org.eclipse.jetty' | |||
relocate 'org.eclipse.jetty.orbit', 'ai.h2o.org.eclipse.jetty.orbit' | |||
relocate 'com.google.common', 'ai.h2o.com.google.common' | |||
|
|||
exclude 'www/flow/packs/test-*/**' |
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 would create JIRA on H2O project, just to let folks know this can be removed
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.
Sounds good to me. @michalkurka WDYT about having the testing flows completely separated from h2o-docs
examples?
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 think that can be done :)
integration tests are failing because they don't have the classes available. |
I think that for now, we can put back the examples as they are really small and we can seperate the jars in the next iterations after release. What do you think @mn-mikke ? |
Ok, I will put them back and create a separate jira for the removal. |
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.
Looks great!
(cherry picked from commit fc8e908)
(cherry picked from commit fc8e908)
(cherry picked from commit fc8e908)
(cherry picked from commit fc8e908)
No description provided.