Merge pull request #3133 from zpao/more-lint

More lint rules, fixes
This commit is contained in:
Paul O’Shannessy
2015-02-12 15:37:22 -08:00
7 changed files with 12 additions and 18 deletions
+2
View File
@@ -25,6 +25,8 @@ rules:
consistent-return: 2
no-unused-vars: [2, args: none]
quotes: [2, 'single']
space-before-blocks: 2
indent: [2, 2, indentSwitchCase: true]
# WARNINGS
# This is the only one that's hard to track since we don't lint just changes.
+1 -1
View File
@@ -328,7 +328,7 @@ var ReactBrowserEventEmitter = assign({}, ReactEventEmitterMixin, {
*
* @see http://www.quirksmode.org/dom/events/scroll.html
*/
ensureScrollValueMonitoring: function(){
ensureScrollValueMonitoring: function() {
if (!isMonitoringScrollValue) {
var refresh = ViewportMetrics.refreshScrollValues;
ReactBrowserEventEmitter.ReactEventListener.monitorScrollValue(refresh);
@@ -11,7 +11,7 @@
'use strict';
var keyOf = require('keyOf');
var keyOf = require('keyOf');
/**
* Module that is injectable into `EventPluginHub`, that specifies a
+2 -2
View File
@@ -132,8 +132,8 @@ var ReactDOMIDOperations = {
* @see {Danger.dangerouslyReplaceNodeWithMarkup}
*/
dangerouslyReplaceNodeWithMarkupByID: function(id, markup) {
var node = ReactMount.getNode(id);
DOMChildrenOperations.dangerouslyReplaceNodeWithMarkup(node, markup);
var node = ReactMount.getNode(id);
DOMChildrenOperations.dangerouslyReplaceNodeWithMarkup(node, markup);
},
/**
@@ -1,4 +1,3 @@
/**
* Copyright 2013-2014, Facebook, Inc.
* All rights reserved.
+5 -12
View File
@@ -1,17 +1,10 @@
/**
* Copyright 2013-2014 Facebook, Inc.
* Copyright 2013-2014, Facebook, Inc.
* All rights reserved.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
* This source code is licensed under the BSD-style license found in the
* LICENSE file in the root directory of this source tree. An additional grant
* of patent rights can be found in the PATENTS file in the same directory.
*
* @providesModule setTextContent
*/
+1 -1
View File
@@ -202,7 +202,7 @@ var ReactCompositeComponentMixin = {
ReactLifeCycle.currentlyMountingInstance = this;
try {
if (inst.componentWillMount) {
inst.componentWillMount();
inst.componentWillMount();
// When mounting, calls to `setState` by `componentWillMount` will set
// `this._pendingStateQueue` without triggering a re-render.
if (this._pendingStateQueue) {