Make global declarations compatible with JSLint and JSHint #68

Merged
merged 1 commit into from Oct 4, 2012

Conversation

Projects
None yet
2 participants
@osener
Contributor

osener commented Oct 4, 2012

Some JavaScript linters don't expect whitespace before the global directive.

http://www.jslint.com/lint.html
http://www.jshint.com/docs/

Make global declarations compatible with JSLint and JSHint
Some JavaScript linters doesn't expect whitespace before the global directive.
@thomblake

This comment has been minimized.

Show comment Hide comment
@thomblake

thomblake Oct 4, 2012

Owner

Wow, that's ugly. Fine.

Do you use a keybinding for that command? I've been meaning to add one.

Owner

thomblake commented Oct 4, 2012

Wow, that's ugly. Fine.

Do you use a keybinding for that command? I've been meaning to add one.

thomblake added a commit that referenced this pull request Oct 4, 2012

Merge pull request #68 from osener/patch-2
Make global declarations compatible with JSLint and JSHint

@thomblake thomblake merged commit af74405 into thomblake:master Oct 4, 2012

@osener

This comment has been minimized.

Show comment Hide comment
@osener

osener Oct 4, 2012

Contributor

Do C-c C-d as a mnemonic for declare global variable and maybe C-c C-g as an easy way to append to js3-additional-externs sound good to you?

Contributor

osener commented Oct 4, 2012

Do C-c C-d as a mnemonic for declare global variable and maybe C-c C-g as an easy way to append to js3-additional-externs sound good to you?

@thomblake

This comment has been minimized.

Show comment Hide comment
@thomblake

thomblake Oct 4, 2012

Owner

Sounds fine to me.

Owner

thomblake commented Oct 4, 2012

Sounds fine to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment