Re: [PR] Add multiple datasources to Jdbc example [camel-quarkus-examples]

2023-11-23 Thread via GitHub
spatnity commented on PR #180: URL: https://github.com/apache/camel-quarkus-examples/pull/180#issuecomment-1824102970 Thanks for the feedback @aldettinger , I will fix them -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] Add multiple datasources to Jdbc example [camel-quarkus-examples]

2023-11-23 Thread via GitHub
aldettinger merged PR #180: URL: https://github.com/apache/camel-quarkus-examples/pull/180 -- 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-unsub

Re: [PR] Add multiple datasources to Jdbc example [camel-quarkus-examples]

2023-11-22 Thread via GitHub
spatnity commented on PR #180: URL: https://github.com/apache/camel-quarkus-examples/pull/180#issuecomment-1823219771 Done! thanks @jamesnetherton -- 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 t

Re: [PR] Add multiple datasources to Jdbc example [camel-quarkus-examples]

2023-11-21 Thread via GitHub
jamesnetherton commented on PR #180: URL: https://github.com/apache/camel-quarkus-examples/pull/180#issuecomment-1821023830 Since there's no Docker on the Windows CI runner, you'll need to add a Maven profile like this: https://github.com/apache/camel-quarkus-examples/blob/main/jpa-

Re: [PR] Add multiple datasources to Jdbc example [camel-quarkus-examples]

2023-11-21 Thread via GitHub
aldettinger commented on PR #180: URL: https://github.com/apache/camel-quarkus-examples/pull/180#issuecomment-1820588678 Well done :+1: Totally forgot about that one. The build looks in good shape let's see. -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] Add multiple datasources to Jdbc example [camel-quarkus-examples]

2023-11-21 Thread via GitHub
spatnity commented on PR #180: URL: https://github.com/apache/camel-quarkus-examples/pull/180#issuecomment-1820581636 I have run `mvn org.l2x6.cq:cq-maven-plugin:0.33.0:update-examples-json` from the project root directory which automatically updated `examples.json` -- This is an automa

Re: [PR] Add multiple datasources to Jdbc example [camel-quarkus-examples]

2023-11-21 Thread via GitHub
aldettinger commented on PR #180: URL: https://github.com/apache/camel-quarkus-examples/pull/180#issuecomment-1820567496 @spatnity Yes, it could be that. Did you needed to update the json file manually ? Or were you able to use a command ? -- This is an automated message from the Apache

Re: [PR] Add multiple datasources to Jdbc example [camel-quarkus-examples]

2023-11-20 Thread via GitHub
jamesnetherton commented on PR #180: URL: https://github.com/apache/camel-quarkus-examples/pull/180#issuecomment-1820367159 You don't need to run `update-examples-json` locally. But you need to ensue that there's an entry for the new project added to [docs/modules/ROOT/attachments/examples

Re: [PR] Add multiple datasources to Jdbc example [camel-quarkus-examples]

2023-11-20 Thread via GitHub
spatnity commented on PR #180: URL: https://github.com/apache/camel-quarkus-examples/pull/180#issuecomment-1820352155 After running this maven goal `org.l2x6.cq:cq-maven-plugin:0.33.0:update-examples-json`, there is a `examples.json` file that is getting created (Untracked) which has nothi

Re: [PR] Add multiple datasources to Jdbc example [camel-quarkus-examples]

2023-11-20 Thread via GitHub
spatnity commented on PR #180: URL: https://github.com/apache/camel-quarkus-examples/pull/180#issuecomment-1820320805 Yeah `README.adoc` syntax used is different compared to others.I will update it -- This is an automated message from the Apache Git Service. To respond to the message, pl

Re: [PR] Add multiple datasources to Jdbc example [camel-quarkus-examples]

2023-11-20 Thread via GitHub
aldettinger commented on PR #180: URL: https://github.com/apache/camel-quarkus-examples/pull/180#issuecomment-1819835277 @spatnity The error message from ci build is: ``` Failed to execute goal org.l2x6.cq:cq-maven-plugin:0.33.0:update-examples-json (default-cli) on proje

Re: [PR] Add multiple datasources to Jdbc example [camel-quarkus-examples]

2023-11-16 Thread via GitHub
spatnity closed pull request #179: Add multiple datasources to Jdbc example URL: https://github.com/apache/camel-quarkus-examples/pull/179 -- 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

Re: [PR] Add multiple datasources to Jdbc example [camel-quarkus-examples]

2023-11-16 Thread via GitHub
aldettinger commented on PR #179: URL: https://github.com/apache/camel-quarkus-examples/pull/179#issuecomment-1814332228 @spatnity , probably it can rebased now, something like: ``` git pull --rebase ``` Execute README instructions to check everything is still fine.

Re: [PR] Add multiple datasources to Jdbc example [camel-quarkus-examples]

2023-11-16 Thread via GitHub
spatnity commented on PR #179: URL: https://github.com/apache/camel-quarkus-examples/pull/179#issuecomment-1814224357 > Maybe we'll wait the 3.6.0.CR1 [pr](https://github.com/apache/camel-quarkus-examples/pull/178) to be merged before merging this one. Yeah sure! -- This is a

Re: [PR] Add multiple datasources to Jdbc example [camel-quarkus-examples]

2023-11-16 Thread via GitHub
aldettinger commented on PR #179: URL: https://github.com/apache/camel-quarkus-examples/pull/179#issuecomment-1814209087 Maybe we'll wait the 3.6.0.CR1 [pr](https://github.com/apache/camel-quarkus-examples/pull/178) to be merged before merging this one. -- This is an automated message f