Lint rule updates

.eslintrc

@@ -24,8 +24,17 @@
  "ignoredNodes": ["ConditionalExpression"]
  }
  ],
+ "keyword-spacing": ["error"],
  "linebreak-style": ["error", "unix"],
- "max-len": ["error", 120, 4],
+ "max-len": [
+ "error",
+ {
+ "code": 100,
+ "tabWidth": 4,
+ "ignoreStrings": true,
+ "ignoreRegExpLiterals": true
+ }
+ ],
  "no-trailing-spaces": ["error", { "skipBlankLines": true }],
  "no-unused-vars": ["error", {"args": "after-used", "varsIgnorePattern": "^_"}],
  "no-use-before-define": ["error"],

blocks/variables.js

@@ -103,7 +103,7 @@ Blockly.Constants.Variables.CUSTOM_CONTEXT_MENU_VARIABLE_GETTER_SETTER_MIXIN = {
  * @this Blockly.Block
  */
  customContextMenu: function(options) {
- if(this.isInFlyout){
+ if (this.isInFlyout){
  return;
  }
  // Getter blocks have the option to create a setter block, and vice versa.

core/block.js

@@ -1197,7 +1197,7 @@ Blockly.Block.prototype.interpolate_ = function(message, args, lastDummyAlign) {
  }
  }
  }
- if(indexCount != args.length) {
+ if (indexCount != args.length) {
  throw new Error('Block "' + this.type + '": ' +
  'Message does not reference all ' + args.length + ' arg(s).');
  }

core/flyout_base.js

@@ -611,7 +611,7 @@ Blockly.Flyout.prototype.createBlock = function(originalBlock) {
  Blockly.Events.setGroup(true);
  Blockly.Events.fire(new Blockly.Events.Create(newBlock));
  // Fire a VarCreate event for each (if any) new variable created.
- for(var i = 0; i < newVariables.length; i++) {
+ for (var i = 0; i < newVariables.length; i++) {
  var thisVariable = newVariables[i];
  Blockly.Events.fire(new Blockly.Events.VarCreate(thisVariable));
  }

core/touch_gesture.js

@@ -269,7 +269,8 @@ Blockly.TouchGesture.prototype.handleTouchMove = function(e) {
  gestureScale * Blockly.TouchGesture.ZOOM_IN_MULTIPLIER :
  gestureScale * Blockly.TouchGesture.ZOOM_OUT_MULTIPLIER;
  var workspace = this.startWorkspace_;
- var position = Blockly.utils.mouseToSvg(e, workspace.getParentSvg(), workspace.getInverseScreenCTM());
+ var position = Blockly.utils.mouseToSvg(
+ e, workspace.getParentSvg(), workspace.getInverseScreenCTM());
  workspace.zoom(position.x, position.y, delta);
  }
  this.previousScale_ = scale;

core/xml.js

@@ -549,7 +549,7 @@ Blockly.Xml.domToBlock = function(xmlBlock, workspace) {
  var newVariables = Blockly.Variables.getAddedVariables(workspace,
  variablesBeforeCreation);
  // Fire a VarCreate event for each (if any) new variable created.
- for(var i = 0; i < newVariables.length; i++) {
+ for (var i = 0; i < newVariables.length; i++) {
  var thisVariable = newVariables[i];
  Blockly.Events.fire(new Blockly.Events.VarCreate(thisVariable));
  }