-
Notifications
You must be signed in to change notification settings - Fork 41
Conversation
private File globalPropertiesFile() { | ||
def env = System.getenv() | ||
String configFolder = env['CONFIG_FOLDER'] | ||
return new File("${configFolder}/common/global.properties") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will it work on Windows?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, it will. You asked a similar question in the past and I have already explained it to you :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't remember. Maybe it wasn't me or maybe I have too many threads to handle I just threw away your explanations from my mind ;). Anyway great that you checked it.
I'm thinking about moving LogbackConfig class to micro-infra-spring. WDYTAI? |
There are no direct dependencies to Logback, so it sound sensible. |
You would move it to which module? |
config, already done in 4finance/micro-infra-spring#330 |
@marcingrzejszczak, I have used logback configuration from micro-infra-spring. Could you please look now? |
+1 |
0b79d8b
to
583ebe7
Compare
No description provided.