2015-08-31 05:31:14 +00:00
|
|
|
# Coding Style
|
|
|
|
|
|
|
|
These are the style guidelines for coding in Electron.
|
2013-09-09 07:35:57 +00:00
|
|
|
|
2013-08-14 22:43:35 +00:00
|
|
|
## C++ and Python
|
|
|
|
|
2015-08-31 05:31:14 +00:00
|
|
|
For C++ and Python, we follow Chromium's [Coding
|
|
|
|
Style](http://www.chromium.org/developers/coding-style). There is also a
|
|
|
|
script `script/cpplint.py` to check whether all files conform.
|
2013-08-14 22:43:35 +00:00
|
|
|
|
2015-08-31 05:31:14 +00:00
|
|
|
The Python version we are using now is Python 2.7.
|
2013-08-14 22:43:35 +00:00
|
|
|
|
2015-08-31 05:31:14 +00:00
|
|
|
The C++ code uses a lot of Chromium's abstractions and types, so it's
|
|
|
|
recommended to get acquainted with them. A good place to start is
|
|
|
|
Chromium's [Important Abstractions and Data Structures](https://www.chromium.org/developers/coding-style/important-abstractions-and-data-structures)
|
|
|
|
document. The document mentions some special types, scoped types (that
|
|
|
|
automatically release their memory when going out of scope), logging mechanisms
|
|
|
|
etc.
|
2015-08-21 19:23:49 +00:00
|
|
|
|
2013-08-14 22:43:35 +00:00
|
|
|
## CoffeeScript
|
|
|
|
|
2013-08-29 14:37:51 +00:00
|
|
|
For CoffeeScript, we follow GitHub's [Style
|
2015-08-31 05:31:14 +00:00
|
|
|
Guide](https://github.com/styleguide/javascript) and the following rules:
|
2013-08-14 22:43:35 +00:00
|
|
|
|
2013-08-29 14:37:51 +00:00
|
|
|
* Files should **NOT** end with new line, because we want to match Google's
|
|
|
|
styles.
|
|
|
|
* File names should be concatenated with `-` instead of `_`, e.g.
|
|
|
|
`file-name.coffee` rather than `file_name.coffee`, because in
|
|
|
|
[github/atom](https://github.com/github/atom) module names are usually in
|
2015-08-31 05:31:14 +00:00
|
|
|
the `module-name` form. This rule only applies to `.coffee` files.
|
2013-08-14 22:43:35 +00:00
|
|
|
|
|
|
|
## API Names
|
|
|
|
|
2013-08-29 14:37:51 +00:00
|
|
|
When creating a new API, we should prefer getters and setters instead of
|
2015-08-31 05:31:14 +00:00
|
|
|
jQuery's one-function style. For example, `.getText()` and `.setText(text)`
|
2013-08-29 14:37:51 +00:00
|
|
|
are preferred to `.text([text])`. There is a
|
2015-08-31 05:31:14 +00:00
|
|
|
[discussion](https://github.com/atom/electron/issues/46) on this.
|