Files
Nikolay Vitkov d76576c6fe [deps] Update EsLint v8.57.1 and plugins
This is need to be able to move to flat config which I am handling
in a separate CL.
The overrides are there as these packages got update today,
so we need to way, but they were last touched 3 years ago,
so reviewing their changes upstream as well.

Bug: none
Change-Id: I61534f881fd4e747c716b42b773f8f7f3e9bc761
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6063618
Commit-Queue: Nikolay Vitkov <nvitkov@chromium.org>
Reviewed-by: Danil Somsikov <dsv@chromium.org>
2024-12-02 14:40:07 +00:00

2.2 KiB

import/first

🔧 This rule is automatically fixable by the --fix CLI option.

This rule reports any imports that come after non-import statements.

Rule Details

import foo from './foo'

// some module-level initializer
initWith(foo)

import bar from './bar' // <- reported

Providing absolute-first as an option will report any absolute imports (i.e. packages) that come after any relative imports:

import foo from 'foo'
import bar from './bar'

import * as _ from 'lodash' // <- reported

If you really want import type ordering, check out import/order.

Notably, imports are hoisted, which means the imported modules will be evaluated before any of the statements interspersed between them. Keeping all imports together at the top of the file may prevent surprises resulting from this part of the spec.

On directives

Directives are allowed as long as they occur strictly before any import declarations, as follows:

'use super-mega-strict'

import { suchFoo } from 'lame-fake-module-name'  // no report here

A directive in this case is assumed to be a single statement that contains only a literal string-valued expression.

'use strict' would be a good example, except that modules are always in strict mode so it would be surprising to see a 'use strict' sharing a file with imports and exports.

Given that, see #255 for the reasoning.

With Fixer

This rule contains a fixer to reorder in-body import to top, the following criteria applied:

  1. Never re-order relative to each other, even if absolute-first is set.
  2. If an import creates an identifier, and that identifier is referenced at module level before the import itself, that won't be re-ordered.

When Not To Use It

If you don't mind imports being sprinkled throughout, you may not want to enable this rule.

Further Reading