Skip to content

Commit

Permalink
Merge pull request #5993 from WordPress/fix/lint-warning
Browse files Browse the repository at this point in the history
Fix findDOMNode lint warning
  • Loading branch information
noisysocks committed Apr 5, 2018
2 parents fc2d87d + 4d5e861 commit 8a95e94
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
3 changes: 3 additions & 0 deletions components/drop-zone/provider.js
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,9 @@ class DropZoneProvider extends Component {
window.addEventListener( 'dragover', this.dragOverListener );
window.addEventListener( 'drop', this.onDrop );
window.addEventListener( 'mouseup', this.resetDragState );

// Disable reason: Can't use a ref since this component just renders its children
// eslint-disable-next-line react/no-find-dom-node
this.container = findDOMNode( this );
}

Expand Down
1 change: 0 additions & 1 deletion eslint/config.js
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,6 @@ module.exports = {
'react/jsx-key': 'error',
'react/jsx-tag-spacing': 'error',
'react/no-children-prop': 'off',
'react/no-find-dom-node': 'warn',
'react/prop-types': 'off',
semi: 'error',
'semi-spacing': 'error',
Expand Down

0 comments on commit 8a95e94

Please sign in to comment.