Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convert from CoffeeScript to JavaScript #4065

Merged
merged 11 commits into from Jan 13, 2016
Merged

Convert from CoffeeScript to JavaScript #4065

merged 11 commits into from Jan 13, 2016

Conversation

kevinsawicki
Copy link
Contributor

This pull request migrates all of the CoffeeScript source files to JavaScript.

πŸ“ˆ CoffeeScript was great when this project started, but many of the best parts of it are now in ES6 and available out of the box in Electron. See this blog post for some examples.

β˜• The JavaScript files in this pull request were generated using CoffeeScript 1.10.0. A second pass will be made in a follow-up pull request to reformat the JavaScript and draft a new styleguide. This is done to minimally disrupt any open or soon to be opened pull requests that may touch the existing .coffee files.

πŸ“ The comments have been retained by first migrating all the # comments to ### which were then compiled to /* */ style comments in the JavaScript.

🎯 The goal is to make Electron as approachable as possible and also to have its source code reflect the capabilities of the V8 version being used.

Changes to be made in subsequent pull requests

  • Cleanup accidental return statements
  • Cleanup comment formatting
  • Cleanup variable scoping
  • Remove CoffeeScript helpers like extend, slice, and indexOf
  • Use let and const appropriately
  • Use arrow (=>) functions

@kevinsawicki kevinsawicki changed the title [WIP] Convert CoffeeScript to JavaScript [WIP] Convert from CoffeeScript to JavaScript Jan 12, 2016
@lukeapage
Copy link
Contributor

woop to this πŸ‘

@kamesh-a
Copy link

πŸ‘

@bennycode
Copy link
Contributor

Keep going' πŸ‘ 😎

@zcbenz
Copy link
Member

zcbenz commented Jan 13, 2016

πŸ‘

zcbenz added a commit that referenced this pull request Jan 13, 2016
[WIP] Convert from CoffeeScript to JavaScript
@zcbenz zcbenz merged commit d9d821c into master Jan 13, 2016
@zcbenz zcbenz deleted the coffee-to-js branch January 13, 2016 03:13
@stevemao
Copy link

I'm so happy for this change πŸ˜„

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants