Github user kkhatua commented on the issue:

    https://github.com/apache/drill/pull/1043
  
    @cgivre 
    Just edited the last comment to remove the +1 [ Learnt that I should wait 
till there is a response to my asks first! :) ] . 
    To summarize, here are my 3 asks:
    1. Change the theme to **Crimson** or **Eclipse** since that's a more 
familiar theme and actually helps the colors stand out.
    2. Change the `src-min-noconflict` directory to reflect the name of the 
library, whose files it contains. e.g. `aceJs`
    3. Remove any non-mandatory files, e.g. `snippets` directory doesn't seems 
to be a requirement for the library. This would help in maintaining future 
updates to the library or debugging. 


---

Reply via email to