pandaapo commented on PR #3988:
URL: https://github.com/apache/eventmesh/pull/3988#issuecomment-1893652061

   > I think we should only use one configuration file, either `eventmesh.yml` 
or `eventmesh.properties`. Having both concurrently would pose difficulties in 
maintenance.
   > 
   > Considering that connectors all use YAML configuration files, the presence 
of `eventmesh.properties` feels inconsistent, since it is one of the few 
properties files in EventMesh. Therefore, my preference would be to standardize 
the use of YAML files.
   > 
   > In our project, we currently have both `.yml` and `.yaml` files, and both 
file extensions are valid. However, it would be necessary to choose one for 
consistency. Although `.yaml` is a more standard and compliant file extension 
compared to `.yml`, the majority of our project uses `.yml`. Hence, I recommend 
standardizing on the `.yml` extension (except for `.asf.yaml`).
   
   Thanks for your review. But sorry, many of your points I couldn't agree 
with, and they were even a bit strange to me. No offense, I'm willing to wait 
for more members of the community to express their views on these points of 
yours. If there are no objections, I will revise.


-- 
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]

Reply via email to