Add Eslint rule to address import overrides of global functions/object
As a result of !24719 (comment 287277100), we should add an eslint rule to prevent importing functions with the same name as available MDN.
Ie throwing an error for the following
import { escape } from 'lodash'
What can potentially happen is if escape
is being used in the code and then the import is removed, we will be using the wrong reference to escape
and potentially open us up to vulnerabilities.
https://eslint.org/docs/rules/no-shadow Seems to be the recommended eslint rule. Given this will throw A LOT of errors, we will want to make sure to only error on escape
for the first iteration.
/cc @pslaughter