Drop CoffeeScript

[lipis]

We decided to slowly start moving away from CoffeeScript to JavaScript! As this step is impossible to do it in one go, here is the list of things needs to be done before we sign it off.

Tools that could help

• js2.coffee
• lebab.io
• decaffeinate

Things to do

• Support JavaScript, along with CoffeeScript in the build scripts (Support JavaScript for writing source files #883)
• Add ESLint to build pipeline (Added ESLint to build pipeline #889)
• app/
  • script/
    • config.coffee (Convert config to ES6 #888)
    • announce/ (ES6: Migrated "announce" #899)
    • assets/ (ES6: Migrated "assets" #912)
    • audio/ (ES6: Migrate "audio" with proper logging #925)
    • auth/ (ES6: Migrated "auth" directory #947)
    • bot/(ES6: migrated "bot" #941)
    • cache/ (ES6: migrated "cache" #946)
    • calling/ (V3 group calls #1125)
      • Remove deprecated calling v2 code (Remove support for v2 calling #1340)
    • client/ (ES6: migrated "client" #976)
    • components/ (Migrate 'components' to es6 #1207)
    • connect/ (ES6: migrated "connect" #1030)
    • conversation/ (ES6 migrate 'conversation' #1172)
    • cryptography/ (ES6: migrated "cryptography" #1048)
    • entity/ (ES6: migrated "entity" (part 2) #1195)
    • entity/message (ES6: migrated "entity/message"  #1097)
    • ephemeral/ (ES6: Migrated ephemeralTimings #950)
    • event/ (ES6: migrated "event" #1078)
    • extension/ (ES6: migrated "extension" #1081)
    • links/ (ES6: migrated "links" #963)
    • localization/ (ES6: migrate "localization" #1036)
    • location/ (ES6: Migrated GeoLocation #951)
    • main/ (ES6: migrated "main" #1084)
    • media/ (ES6: migrated "media" #1065)
    • message/ (ES6: migrated "message" #1086)
    • properties/ (ES6: migrated "properties" #1079)
    • search/ (ES6: migrated "search" #1082)
    • service/ (ES6: migrated "service" #1080)
    • storage/ (ES6: migrated "storage" #1083)
    • system_notification/ (ES6: migrated "system_notification" #1052)
    • telemetry/ (ES6: migrated "telemetry" #1089)
    • tracking/ (ES6: migrated "tracking" #1085)
    • ui/ (ES6: migrated "ui" #1120)
    • user/ (ES6: migrated "user" #1138)
    • util/ (ES6: migrated "util" #1114)
    • view_model/ (ES6: Migrate 'view_model' #1236)
      • view_model/bindings/ (ES6 migrate 'view_model/bindings' #1222)
      • view_model/content/ (ES6: Migrate 'view_model/content' #1235)
      • view_model/list/ (ES6: Migrate 'view_model/list' #1233)
  • worker (ES6 migrate workers #1374)
• test/api
• test/unit_tests/ (Migrate 'test/unit_tests' to ES6 #1201)
  • announce/ (Migrate 'test/unit_tests' to ES6 #1201)
  • assets/ (Migrate 'test/unit_tests' to ES6 #1201)
  • audio/ (Migrate 'test/unit_tests' to ES6 #1201)
  • auth/ (Migrate 'test/unit_tests' to ES6 #1201)
  • cache/ (Migrate 'test/unit_tests' to ES6 #1201)
  • calling/ (Migrate 'test/unit_tests' to ES6 #1201)
  • client/ (Migrate 'test/unit_tests' to ES6 #1201)
  • connect/ (Migrate 'test/unit_tests' to ES6 #1201)
  • conversation/ (Migrate 'test/unit_tests' to ES6 #1201)
  • cryptography/ (Migrate 'test/unit_tests' to ES6 #1201)
  • entity/ (Migrate 'test/unit_tests' to ES6 #1201)
  • event/ (Migrate 'test/unit_tests' to ES6 #1201)
  • extension/ (Migrate 'test/unit_tests' to ES6 #1201)
  • links/ (Migrate 'test/unit_tests' to ES6 #1201)
  • localization/ (ES6: migrate "localization" #1036)
  • location/ (ES6: Migrated GeoLocation #951)
  • media/ (Migrate 'test/unit_tests' to ES6 #1201)
  • message/ (Migrate 'test/unit_tests' to ES6 #1201)
  • search/ (Migrate 'test/unit_tests' to ES6 #1201)
  • service/ (Migrate 'test/unit_tests' to ES6 #1201)
  • storage/ (Migrate 'test/unit_tests' to ES6 #1201)
  • system_notification/ (Migrate 'test/unit_tests' to ES6 #1201)
  • tracking/ (Migrate 'test/unit_tests' to ES6 #1201)
  • ui/ (Migrate 'test/unit_tests' to ES6 #1201)
  • user/ (Migrate 'test/unit_tests' to ES6 #1201)
  • util/ (Migrate 'test/unit_tests' to ES6 #1201)
  • view_model/ (Migrate 'test/unit_tests' to ES6 #1201)
• Gruntfile.coffee (Gruntfile.coffee → Gruntfile.js #885)
• grunt/config/ (ES6 migrate grunt #1376)
• grunt/tasks/ (ES6 migrate grunt #1376)
• Make sure there are no *.coffee files in the project
• Remove build tasks package.json
• Remove the CoffeeScript dependency from package.json

[balupton]

Not sure what the reasoning is, but CoffeeScript v2 will be out soon: coffeescript6/discuss#71

Over at Bevry, we moved from CoffeeScript to ESNext for all our things throughout 2016. Our reasoning is at: https://discuss.bevry.me/t/move-from-coffeescript-to-es6/30/1 - perhaps will have some useful resources for your own move.

[herrmannplatz]

Thanks for sharing this. Haven't followed the whole v2 discussion. Was quite surprised to see that they actually started working on this finally. But this doesn't change our decision. What you mentioned is similar to what we discussed. Besides that some more reasons from our site are:

• We only support recent browser versions which can handle es6. So we can get rid of a compile step and start writing plain es6. Using babel on top is also easier with this setup.
• We had some nasty bugs because of implicit returns
• Interoperability with other frameworks and tooling. People putting less and less effort to support CoffeeScript (e.g React esp. JSX)

To sum it up. CoffeeScript introduced some really helpful concepts and made coding more convenient, but JavaScript nowadays offers so much more.

[lipis]

[lipis]

[lipis]

[lipis]

[lipis]

[gregor]

[bennycode]

[bennycode]

[gregor]

Finally 😄

[bennycode]

[bennycode]

[lipis]

[gregor]

[lipis]

[gregor]

[gregor]

Another milestone passed 🎉

[gregor]

[gregor]

Bääm

[herrmannplatz]

[herrmannplatz]

DONE!

[lipis]

KAPUT!

[herrmannplatz]

Now i should start drinking less coffee -_-'

[lipis]

but but but Java is Coffee 🗡

[gregor]

Let's have a celebration tonight 🎉