imbajin commented on PR #2438: URL: https://github.com/apache/incubator-hugegraph/pull/2438#issuecomment-1974797568
> > * [x] keep the `dist/*/bin/*.properties` (user config files) clean (no need to add header here) > > @imbajin Is it only for cleaning the license header in **properties** files in user configuration files here? I found that some other user configuration files have license headers. > >  > > Also, do **properties** files in the `conf-raft*` directory need to have their license headers cleaned? > >  @VGalaxies The current judgment points are: 1. If a configuration file is not **frequently used/modified** by **users**, following ASF's suggestion may be better to add a file header to reduce disputes 2. In HG, considering that there are already many configuration files and the content is relatively long, the commonly used configurations by users are not separately added to avoid affecting the **user experience** 3. (like point 1) Configuration files from testing/and other modules could also add headers (although not required) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
