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

[WINDUPRULE-432,433,435,436,437,438] - camel components moved out of camel-core #401

Merged
merged 8 commits into from Feb 27, 2020

Conversation

@windupgithubbot1
Copy link

Can one of the admins verify this patch?

@mrizzi
Copy link
Member

mrizzi commented Jan 30, 2020

test this please

Copy link
Member

@mrizzi mrizzi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel like this PR depends on #400 to have the xml-moved-components-0000[0-2] rules done with tests.

@mmelko mmelko changed the title [WINDUPRULE-437] - camel-rest moved out of camel-core DO NOT MERGE - [WINDUPRULE-437,432,433,435,436,437,438] - camel components moved out of camel-core Feb 19, 2020
@mmelko mmelko force-pushed the WINDUPRULE-437-camel-rest-moved branch from ffd775f to 425cf9e Compare February 19, 2020 12:49
@mmelko mmelko changed the title DO NOT MERGE - [WINDUPRULE-437,432,433,435,436,437,438] - camel components moved out of camel-core DO NOT MERGE - [WINDUPRULE-432,433,435,436,437,438] - camel components moved out of camel-core Feb 19, 2020
@mmelko mmelko changed the title DO NOT MERGE - [WINDUPRULE-432,433,435,436,437,438] - camel components moved out of camel-core [WINDUPRULE-432,433,435,436,437,438] - camel components moved out of camel-core Feb 19, 2020
@mmelko
Copy link
Contributor Author

mmelko commented Feb 19, 2020

retest

@mrizzi
Copy link
Member

mrizzi commented Feb 19, 2020

add to whitelist

@mmelko
Copy link
Contributor Author

mmelko commented Feb 20, 2020

Please remove label "Depend on another PR". PR is already merged to master.

</not>
</when>
<perform>
<fail message="[xml-moved-components] camel-rest dependency moved hint was not found!"/>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<fail message="[xml-moved-components] camel-rest dependency moved hint was not found!"/>
<fail message="[xml-moved-components] camel-attachments dependency moved hint was not found!"/>

</not>
</when>
<perform>
<fail message="[xml-moved-components] camel-rest dependency moved hint was not found!"/>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<fail message="[xml-moved-components] camel-rest dependency moved hint was not found!"/>
<fail message="[xml-moved-components] camel-dataformat dependency moved hint was not found!"/>

</not>
</when>
<perform>
<fail message="[xml-moved-components] `camel-zip-deflater` dependency moved hint was not found!"/>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<fail message="[xml-moved-components] `camel-zip-deflater` dependency moved hint was not found!"/>
<fail message="[xml-moved-components] `camel-xpath` dependency moved hint was not found!"/>

<iteration over="dependencies-block">
<hint title="`camel-zip-deflater` component has been moved" effort="1" category-id="mandatory">
<message>`camel-zip-deflater` dependency required. Component has been moved from `camel-core` to separate artifact
org.apache.camel:camel-dataformat that has to be added as a dependency to your project pom.xml file
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
org.apache.camel:camel-dataformat that has to be added as a dependency to your project pom.xml file
org.apache.camel:camel-zip-deflater that has to be added as a dependency to your project pom.xml file

<iteration over="dependencies-block">
<hint title="`camel-zip-deflater` component has been moved" effort="1" category-id="mandatory">
<message>`camel-zip-deflater` dependency required. Component has been moved from `camel-core` to separate artifact
org.apache.camel:camel-dataformat that has to be
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
org.apache.camel:camel-dataformat that has to be
org.apache.camel:camel-zip-deflator that has to be

<iteration over="dependencies-block">
<hint title="`camel-xpath` component has been moved" effort="1" category-id="mandatory">
<message>`camel-xpath` dependency required. Component has been moved from `camel-core` to separate artifact
org.apache.camel:camel-dataformat that has to be added as a dependency to your project pom.xml file
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
org.apache.camel:camel-dataformat that has to be added as a dependency to your project pom.xml file
org.apache.camel:camel-xpath that has to be added as a dependency to your project pom.xml file

@mmelko mmelko force-pushed the WINDUPRULE-437-camel-rest-moved branch from 425cf9e to 559d9e3 Compare February 26, 2020 12:06
@mmelko mmelko requested a review from m-brophy February 26, 2020 12:58
@mmelko
Copy link
Contributor Author

mmelko commented Feb 26, 2020

@m-brophy thanks for the review. All changes should be included now.

Copy link
Contributor

@m-brophy m-brophy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @mmelko, changes approved.

@m-brophy m-brophy merged commit b10bf85 into windup:master Feb 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants