matthiasblaesing commented on PR #9289:
URL: https://github.com/apache/netbeans/pull/9289#issuecomment-4183252588

   > I will have a second look to see if matching ".env." is enough for various 
frameworks. But this might require double MIME registration to fix the ".test" 
mime type association.
   
   Not sure that I get what you mean. `test.env.js` was just an illustation. 
The file could also be `siteA.env.js`, `staging.env.js` and so on. The filetype 
could also be `.ts` (I'm aware of the convention to call the file 
`environment.ts` in angular, but they could also be named `staging.env.ts`, 
`.java` or any other filetype.
   
   The problem with .env files from my POV is, that they are an ad-hoc solution 
combined with a bad naming convention. From my POV it would much more sense to 
call the shown .env files `dev.env` `dist.env` and `local.env`. That way the OS 
won't hide the sample files and filetype matching is trivial.
   
   What might be acceptable for the substring matcher would be to move the 
mimetype mapping far down and let it be considered last/late. That might need 
adjustment to other modules (the LSP integrations currently have a very high 
position, so this might need to be lowered).


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

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists

Reply via email to