@uppy/robodog: fix linter warnings (#3808)

packages/@uppy/robodog/src/TransloaditResultsPlugin.js

@@ -26,9 +26,9 @@ class TransloaditResultsPlugin extends BasePlugin {
     assemblies.forEach((assembly) => {
       Object.keys(assembly.results).forEach((stepName) => {
         const results = assembly.results[stepName]
-        results.forEach((result) => {
+        results.forEach((resultObject) => {
           assemblyResults.push({
-            ...result,
+            ...resultObject,
             assemblyId: assembly.assembly_id,
             stepName,
           })

packages/@uppy/robodog/src/addProviders.js

@@ -1,3 +1,4 @@
+/* eslint-disable global-require */
 const Transloadit = require('@uppy/transloadit')
 const has = require('@uppy/utils/lib/hasProperty')
 
@@ -39,6 +40,7 @@ function addRemoteProvider (uppy, name, opts) {
     companionAllowedHosts: Transloadit.COMPANION_PATTERN,
   }
 
+  // eslint-disable-next-line no-shadow
   remoteProviderOptionNames.forEach((name) => {
     if (has(opts, name)) providerOptions[name] = opts[name]
   })
@@ -68,6 +70,7 @@ function addLocalProvider (uppy, name, opts) {
   const Provider = localProviders[name]
   const providerOptions = {}
 
+  // eslint-disable-next-line no-shadow
   localProviderOptionNames.forEach((name) => {
     if (has(opts, name)) providerOptions[name] = opts[name]
   })

packages/@uppy/robodog/src/form.js

@@ -24,6 +24,7 @@ function mergeDefaultLocale (defaults, userProvided = {}) {
 function form (target, opts) {
   if (!opts) throw new TypeError('robodog.form: must provide an options object')
 
+  // eslint-disable-next-line no-param-reassign
   opts = {
     ...opts,
     locale: mergeDefaultLocale(defaultLocaleStrings, opts.locale),

packages/@uppy/robodog/src/index.js

@@ -8,5 +8,7 @@ module.exports = {
   form,
   pick,
   upload,
+  // We need to keep the require here because we're using `babel-plugin-inline-package-json`.
+  // eslint-disable-next-line global-require
   VERSION: require('../package.json').version,
 }