[GitHub] [camel-k] astefanutti commented on pull request #1816: Add option to save transitive dependencies into custom location

2021-03-01 Thread GitBox
astefanutti commented on pull request #1816: URL: https://github.com/apache/camel-k/pull/1816#issuecomment-787751588 I think it's been superseded by #2048. Feel free to re-open if that is not the case. This is an automated

[GitHub] [camel-k] astefanutti commented on pull request #1816: Add option to save transitive dependencies into custom location

2021-01-21 Thread GitBox
astefanutti commented on pull request #1816: URL: https://github.com/apache/camel-k/pull/1816#issuecomment-763896401 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [camel-k] astefanutti commented on pull request #1816: Add option to save transitive dependencies into custom location

2021-01-21 Thread GitBox
astefanutti commented on pull request #1816: URL: https://github.com/apache/camel-k/pull/1816#issuecomment-764797344 @doru1004 thanks, that looks good to me from an end-user standpoint. From the implementation, I would suggest to try reusing the builder API for local operations as

[GitHub] [camel-k] astefanutti commented on pull request #1816: Add option to save transitive dependencies into custom location

2021-01-21 Thread GitBox
astefanutti commented on pull request #1816: URL: https://github.com/apache/camel-k/pull/1816#issuecomment-764637916 @doru1004 may be worth you having a look at #1816. This is an automated message from the Apache Git

[GitHub] [camel-k] astefanutti commented on pull request #1816: Add option to save transitive dependencies into custom location

2021-01-21 Thread GitBox
astefanutti commented on pull request #1816: URL: https://github.com/apache/camel-k/pull/1816#issuecomment-764488173 @doru1004 thanks, it seems you're a lot more capable than me to formalise what's in my mind ! This is an

[GitHub] [camel-k] astefanutti commented on pull request #1816: Add option to save transitive dependencies into custom location

2021-01-20 Thread GitBox
astefanutti commented on pull request #1816: URL: https://github.com/apache/camel-k/pull/1816#issuecomment-763896401 Damn, I'm not sure what's going on in my brain, but it seems it cannot get that `kamel inspect` does not retrieve details about the Integration resource from the cluster

[GitHub] [camel-k] astefanutti commented on pull request #1816: Add option to save transitive dependencies into custom location

2021-01-20 Thread GitBox
astefanutti commented on pull request #1816: URL: https://github.com/apache/camel-k/pull/1816#issuecomment-763853474 @nicolaferraro what do you think about grouping all the _local-related_ operations into the `local` parent command? I see the downloading of dependencies as an

[GitHub] [camel-k] astefanutti commented on pull request #1816: Add option to save transitive dependencies into custom location

2020-12-03 Thread GitBox
astefanutti commented on pull request #1816: URL: https://github.com/apache/camel-k/pull/1816#issuecomment-738083443 Ah right. For some reasons I thought the `inspect` sub-command could also take an integration name. Out of curiosity, is the `inspect` sub-command meant to have a

[GitHub] [camel-k] astefanutti commented on pull request #1816: Add option to save transitive dependencies into custom location

2020-12-03 Thread GitBox
astefanutti commented on pull request #1816: URL: https://github.com/apache/camel-k/pull/1816#issuecomment-737734726 @doru1004 thanks. As this performs a local build, I suggest to move the logic in the `local` subcommand. I think the later is better suited to group all the subcommands

[GitHub] [camel-k] astefanutti commented on pull request #1816: Add option to save transitive dependencies into custom location

2020-11-24 Thread GitBox
astefanutti commented on pull request #1816: URL: https://github.com/apache/camel-k/pull/1816#issuecomment-732736420 Could you please explicit the use cases you have in mind that would leverage this? I'm wondering whether the `kamel inspect integration` is the right host for this