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/02 16:50:08 UTC
[GitHub] [camel-quarkus] aldettinger opened a new issue #1676: mvn -N cq:promote not to generate an empty application.properties file
aldettinger opened a new issue #1676:
URL: https://github.com/apache/camel-quarkus/issues/1676
----------------------------------------------------------------
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-quarkus] ppalaga commented on issue #1676: mvn -N cq:promote not to generate an empty application.properties file
Posted by GitBox <gi...@apache.org>.
ppalaga commented on issue #1676:
URL: https://github.com/apache/camel-quarkus/issues/1676#issuecomment-689485146
Hm... this is certainly a valid issue, but I must say I find it handy to have the file when I want to add something into it.
How about modifying the formatting mojo (that people should invoke before sending a PR anyway) to remove empty property files?
----------------------------------------------------------------
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-quarkus] aldettinger commented on issue #1676: mvn -N cq:promote not to generate an empty application.properties file
Posted by GitBox <gi...@apache.org>.
aldettinger commented on issue #1676:
URL: https://github.com/apache/camel-quarkus/issues/1676#issuecomment-689540323
Yes, that would make sense. I don't see any case where keeping a properties file with just header comment would be needed.
----------------------------------------------------------------
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-quarkus] ppalaga closed issue #1676: Make mvn -N cq:format remove empty application.properties files
Posted by GitBox <gi...@apache.org>.
ppalaga closed issue #1676:
URL: https://github.com/apache/camel-quarkus/issues/1676
----------------------------------------------------------------
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-quarkus] aldettinger commented on issue #1676: mvn -N cq:promote not to generate an empty application.properties file
Posted by GitBox <gi...@apache.org>.
aldettinger commented on issue #1676:
URL: https://github.com/apache/camel-quarkus/issues/1676#issuecomment-685863504
The application.properties file may not be needed most of the time, more context [here](https://github.com/apache/camel-quarkus/pull/1674#discussion_r482195714).
----------------------------------------------------------------
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