[GitHub] [logging-log4cxx] craterone commented on pull request #72: add color to log4cxx

2021-09-18 Thread GitBox
craterone commented on pull request #72: URL: https://github.com/apache/logging-log4cxx/pull/72#issuecomment-922405875 > Looks good to me, but I agree with Thorsten that at least some documentation is needed(currently, the documentation for the conversion characters is in the documentation

[GitHub] [logging-log4cxx] rm5248 commented on pull request #72: add color to log4cxx

2021-09-18 Thread GitBox
rm5248 commented on pull request #72: URL: https://github.com/apache/logging-log4cxx/pull/72#issuecomment-922340153 Looks good to me, but I agree with Thorsten that at least some documentation is needed(currently, the documentation for the conversion characters is in the documentation for

[GitHub] [logging-log4cxx] ams-tschoening commented on pull request #72: add color to log4cxx

2021-09-18 Thread GitBox
ams-tschoening commented on pull request #72: URL: https://github.com/apache/logging-log4cxx/pull/72#issuecomment-922244507 It would be great to not only put a bunch of code here, but as well describe your use-case, the chosen implementation, limitations, how things are used in the end etc