You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "ppalaga (via GitHub)" <gi...@apache.org> on 2023/05/05 14:38:46 UTC

[GitHub] [camel-quarkus] ppalaga commented on pull request #4873: Snmp - added doc and refactor of the tests

ppalaga commented on PR #4873:
URL: https://github.com/apache/camel-quarkus/pull/4873#issuecomment-1536361813

   @JiriOndrusek thanks for adressing the suggestions!
   
   I dared to fixup some small glitches in this PR by creating a new PR to avoid another review round: https://github.com/apache/camel-quarkus/pull/4876
   
   First, `snm` in the `org.apache.camel.quarkus.component.snm.graal` package name looked like a typo. I changed it to simply `org.apache.camel.quarkus.component.snmp` because we tend to use graal packages for substitutions. I also moved the camel classes to `org.apache.camel.quarkus.component.snmp` because I think there is no reason to introduce a duplicate package. 
   Other than that, i rephrased the docs text a bit. I hope I have not broken anything. 


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