diff options
author | Richard Macklin <richard.github@nrm.com> | 2017-12-11 14:44:52 +0800 |
---|---|---|
committer | Richard Macklin <richard.github@nrm.com> | 2018-11-02 08:40:35 -0700 |
commit | 403c001c56e3980e624da2cb1e1e98d667499d40 (patch) | |
tree | fcdcb33626125f00dff1fc11a9447995f8b7abf4 /activejob/test/jobs/gid_job.rb | |
parent | 7b0b37240a20c74197408ac3b53519e7e18347e9 (diff) | |
download | rails-403c001c56e3980e624da2cb1e1e98d667499d40.tar.gz rails-403c001c56e3980e624da2cb1e1e98d667499d40.tar.bz2 rails-403c001c56e3980e624da2cb1e1e98d667499d40.zip |
Run decaffeinate on action_cable/*.js
Using [decaffeinate], we have converted these files from coffeescript
syntax to ES2015 syntax. Decaffeinate is very conservative in the
conversion process to ensure exact coffeescript semantics are preserved.
Most of the time, it's safe to clean up the code, and decaffeinate has
left suggestions regarding potential cleanups we can take. I'll tackle
those cleanups separately.
After running decaffeinate, I ran:
```
eslint --fix app/javascript
```
using the eslint configuration from ActiveStorage to automatically
correct lint violations in the decaffeinated output. This removed 189
extra semicolons and changed one instance of single quotes to double
quotes.
Note: decaffeinate and eslint can't parse ERB syntax. So I worked around
that by temporarily quoting the ERB:
```diff
@ActionCable =
- INTERNAL: <%= ActionCable::INTERNAL.to_json %>
+ INTERNAL: "<%= ActionCable::INTERNAL.to_json %>"
WebSocket: window.WebSocket
logger: window.console
```
and then removing those quotes after running decaffeinate and eslint.
[decaffeinate]: https://github.com/decaffeinate/decaffeinate
Diffstat (limited to 'activejob/test/jobs/gid_job.rb')
0 files changed, 0 insertions, 0 deletions