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 2022/04/04 13:43:12 UTC

[GitHub] [camel] essobedo opened a new pull request, #7349: CAMEL-17792: Add documentation about the message headers (P-Q)

essobedo opened a new pull request, #7349:
URL: https://github.com/apache/camel/pull/7349

   https://issues.apache.org/jira/browse/CAMEL-17792
   
   ## Motivation
   
   It is now possible to document a message header using annotations, let's leverage it to document the headers of the components.
   
    ## Modifications
   
   For all the components whose name starts with a P or Q
   
   * Document the messages headers using the annotations
   * Migrate the existing documentation of message headers to use the annotations instead
   * In case of common headers class for several endpoint, replace the scheme of each endpoint with a constant to facilitate usage of the element `applicableFor`
   * Move constants corresponding to headers to the headers class.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

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

[GitHub] [camel] davsclaus commented on pull request #7349: CAMEL-17792: Add documentation about the message headers (P-Q)

Posted by GitBox <gi...@apache.org>.
davsclaus commented on PR #7349:
URL: https://github.com/apache/camel/pull/7349#issuecomment-1087591708

   You are getting closer to Z ;)


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

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

[GitHub] [camel] github-actions[bot] commented on pull request #7349: CAMEL-17792: Add documentation about the message headers (P-Q)

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #7349:
URL: https://github.com/apache/camel/pull/7349#issuecomment-1087575203

   :warning: This PR changes Camel components and will be tested automatically.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

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

[GitHub] [camel] davsclaus commented on pull request #7349: CAMEL-17792: Add documentation about the message headers (P-Q)

Posted by GitBox <gi...@apache.org>.
davsclaus commented on PR #7349:
URL: https://github.com/apache/camel/pull/7349#issuecomment-1087739549

   yes this is good


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

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

[GitHub] [camel] github-actions[bot] commented on pull request #7349: CAMEL-17792: Add documentation about the message headers (P-Q)

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #7349:
URL: https://github.com/apache/camel/pull/7349#issuecomment-1087631133

   :no_entry_sign: There are too many components to be tested in this PR, components were removed or the code needs a rebase: (12 likely to be tested)


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

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

[GitHub] [camel] essobedo merged pull request #7349: CAMEL-17792: Add documentation about the message headers (P-Q)

Posted by GitBox <gi...@apache.org>.
essobedo merged PR #7349:
URL: https://github.com/apache/camel/pull/7349


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

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

[GitHub] [camel] essobedo commented on pull request #7349: CAMEL-17792: Add documentation about the message headers (P-Q)

Posted by GitBox <gi...@apache.org>.
essobedo commented on PR #7349:
URL: https://github.com/apache/camel/pull/7349#issuecomment-1087635898

   > You are getting closer to Z ;)
   
   I hope, I will be able to finish this week. It is a very long and laborious task. I'm looking forward finishing it to be able to do something else 😄 
   
   BTW,  for some components, I moved directly the constants from other classes (like the for example the EntryPoint class) to the Constant class while maybe the proper way was to duplicate the constants and mark the constants from other classes as deprecated, WDYT?


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

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