You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2020/09/18 16:33:55 UTC

[GitHub] [camel] bedlaj opened a new pull request #4251: CAMEL-15552: Print dirty files in workflow step

bedlaj opened a new pull request #4251:
URL: https://github.com/apache/camel/pull/4251


   If this step fails, it is currently missing information about the reason. Switched to my fork of has-changes-action temporarilly until https://github.com/UnicornGlobal/has-changes-action/pull/2 gets merged. 
   
   Sample output - https://github.com/bedlaj/camel/runs/1134623405 :
   ```
   Maven build will override some files, which are not commited as part of this PR. Please run maven build and commit generated sources.
    M catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/docs/box-component.adoc
    M catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/docs/twilio-component.adoc
    M catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/others/microprofile-fault-tolerance.json
    M components/camel-box/camel-box-component/src/generated/resources/org/apache/camel/component/box/box.json
    M components/camel-box/camel-box-component/src/main/docs/box-component.adoc
    M components/camel-twilio/src/generated/resources/org/apache/camel/component/twilio/twilio.json
    M components/camel-twilio/src/main/docs/twilio-component.adoc
    M docs/components/modules/ROOT/pages/box-component.adoc
    M docs/components/modules/ROOT/pages/twilio-component.adoc
   ##[error]Process completed with exit code 1.
   ```
   
   [ ] Make sure there is a [JIRA issue](https://issues.apache.org/jira/browse/CAMEL) filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
   [ ] Each commit in the pull request should have a meaningful subject line and body.
   [ ] If you're unsure, you can format the pull request title like `[CAMEL-XXX] Fixes bug in camel-file component`, where you replace `CAMEL-XXX` with the appropriate JIRA issue.
   [ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
   [ ] Run `mvn clean install -Psourcecheck` in your module with source check enabled to make sure basic checks pass and there are no checkstyle violations. A more thorough check will be performed on your pull request automatically.
   Below are the contribution guidelines:
   https://github.com/apache/camel/blob/master/CONTRIBUTING.md


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [camel] bedlaj commented on pull request #4251: CAMEL-15552: Print dirty files in workflow step

Posted by GitBox <gi...@apache.org>.
bedlaj commented on pull request #4251:
URL: https://github.com/apache/camel/pull/4251#issuecomment-694997199


   I am looking if there is some solution to make build faster, but chances are low, it seems this is already extremely well optimized.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [camel] oscerd commented on pull request #4251: CAMEL-15552: Print dirty files in workflow step

Posted by GitBox <gi...@apache.org>.
oscerd commented on pull request #4251:
URL: https://github.com/apache/camel/pull/4251#issuecomment-694967365


   Thanks for this, we need to optimize a bit the gh action in all the Camel ecosystem. Here it would be super cool to being able to test a PR, but the build takes too much.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [camel] bedlaj merged pull request #4251: CAMEL-15552: Print dirty files in workflow step

Posted by GitBox <gi...@apache.org>.
bedlaj merged pull request #4251:
URL: https://github.com/apache/camel/pull/4251


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org