search

Qusic / atom-youcompleteme

YouCompleteMe for Atom Editor
MIT License
82 stars 34 forks source link

Changes required for upcoming CoffeeScript upgrade #87

Closed Ingramz closed 7 years ago

Ingramz commented 7 years ago

Hello!

In Atom 1.23.0, the CoffeeScript dependency will be upgraded from version 1.11.1 to 1.12.7. To ensure that all packages will remain working the same way, we searched for CoffeeScript compile output differences from all Atom packages and found your package to be affected by the update.

This pull request implements the fix mentioned in https://github.com/jashkenas/coffeescript/issues/4760#issuecomment-340001235, which is compatible with both versions.

It could be that the code will remain working despite the compiled output takes slightly different approach. If you believe this is the case, feel free to just close this pull request and consider this just as a heads-up.

Please let me know if you have any questions.

Qusic commented 7 years ago

This fix is not necessary in this case. Thanks for the information anyway!