Skip to content

Commit faee198

Browse files
mskeltonmskelton
authored
chore: Convert internal build to ESM (#331)
1 parent 4c61256 commit faee198

File tree

109 files changed

+309
-299
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

109 files changed

+309
-299
lines changed

eslint.config.mjs renamed to eslint.config.js

File renamed without changes.

‎index.cjs‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
// eslint-disable-next-line @typescript-eslint/no-require-imports
2+
module.exports = require('./dist/index.cjs')

index.d.mts renamed to index.d.ts

File renamed without changes.

‎index.js‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
import plugin from './dist/index.cjs'
2+
3+
export default plugin

‎index.mjs‎

Lines changed: 0 additions & 3 deletions
This file was deleted.

‎package.json‎

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -15,19 +15,20 @@
1515
"engines": {
1616
"node": ">=16.6.0"
1717
},
18-
"type": "commonjs",
19-
"types": "./index.d.mts",
18+
"type": "module",
19+
"types": "./index.d.ts",
2020
"exports": {
2121
".": {
22-
"types": "./index.d.mts",
23-
"import": "./index.mjs",
24-
"require": "./dist/index.js"
22+
"types": "./index.d.ts",
23+
"import": "./index.js",
24+
"require": "./index.cjs"
2525
}
2626
},
2727
"files": [
2828
"dist",
29-
"index.mjs",
30-
"index.d.mts"
29+
"index.js",
30+
"index.cjs",
31+
"index.d.ts"
3132
],
3233
"scripts": {
3334
"build": "tsup src/index.ts --format cjs --out-dir dist",

‎src/index.ts‎

Lines changed: 50 additions & 48 deletions
Original file line numberDiff line numberDiff line change
@@ -1,52 +1,52 @@
11
import globals from 'globals'
2-
import expectExpect from './rules/expect-expect'
3-
import maxExpects from './rules/max-expects'
4-
import maxNestedDescribe from './rules/max-nested-describe'
5-
import missingPlaywrightAwait from './rules/missing-playwright-await'
6-
import noCommentedOutTests from './rules/no-commented-out-tests'
7-
import noConditionalExpect from './rules/no-conditional-expect'
8-
import noConditionalInTest from './rules/no-conditional-in-test'
9-
import noDuplicateHooks from './rules/no-duplicate-hooks'
10-
import noElementHandle from './rules/no-element-handle'
11-
import noEval from './rules/no-eval'
12-
import noFocusedTest from './rules/no-focused-test'
13-
import noForceOption from './rules/no-force-option'
14-
import noGetByTitle from './rules/no-get-by-title'
15-
import noHooks from './rules/no-hooks'
16-
import noNestedStep from './rules/no-nested-step'
17-
import noNetworkidle from './rules/no-networkidle'
18-
import noNthMethods from './rules/no-nth-methods'
19-
import noPagePause from './rules/no-page-pause'
20-
import noRawLocators from './rules/no-raw-locators'
21-
import noRestrictedMatchers from './rules/no-restricted-matchers'
22-
import noSkippedTest from './rules/no-skipped-test'
23-
import noStandaloneExpect from './rules/no-standalone-expect'
24-
import noUnsafeReferences from './rules/no-unsafe-references'
25-
import noUselessAwait from './rules/no-useless-await'
26-
import noUselessNot from './rules/no-useless-not'
27-
import noWaitForSelector from './rules/no-wait-for-selector'
28-
import noWaitForTimeout from './rules/no-wait-for-timeout'
29-
import preferComparisonMatcher from './rules/prefer-comparison-matcher'
30-
import preferEqualityMatcher from './rules/prefer-equality-matcher'
31-
import preferHooksInOrder from './rules/prefer-hooks-in-order'
32-
import preferHooksOnTop from './rules/prefer-hooks-on-top'
33-
import preferLocator from './rules/prefer-locator'
34-
import preferLowercaseTitle from './rules/prefer-lowercase-title'
35-
import preferNativeLocators from './rules/prefer-native-locators'
36-
import preferStrictEqual from './rules/prefer-strict-equal'
37-
import preferToBe from './rules/prefer-to-be'
38-
import preferToContain from './rules/prefer-to-contain'
39-
import preferToHaveCount from './rules/prefer-to-have-count'
40-
import preferToHaveLength from './rules/prefer-to-have-length'
41-
import preferWebFirstAssertions from './rules/prefer-web-first-assertions'
42-
import requireHook from './rules/require-hook'
43-
import requireSoftAssertions from './rules/require-soft-assertions'
44-
import requireToThrowMessage from './rules/require-to-throw-message'
45-
import requireTopLevelDescribe from './rules/require-top-level-describe'
46-
import validDescribeCallback from './rules/valid-describe-callback'
47-
import validExpect from './rules/valid-expect'
48-
import validExpectInPromise from './rules/valid-expect-in-promise'
49-
import validTitle from './rules/valid-title'
2+
import expectExpect from './rules/expect-expect.js'
3+
import maxExpects from './rules/max-expects.js'
4+
import maxNestedDescribe from './rules/max-nested-describe.js'
5+
import missingPlaywrightAwait from './rules/missing-playwright-await.js'
6+
import noCommentedOutTests from './rules/no-commented-out-tests.js'
7+
import noConditionalExpect from './rules/no-conditional-expect.js'
8+
import noConditionalInTest from './rules/no-conditional-in-test.js'
9+
import noDuplicateHooks from './rules/no-duplicate-hooks.js'
10+
import noElementHandle from './rules/no-element-handle.js'
11+
import noEval from './rules/no-eval.js'
12+
import noFocusedTest from './rules/no-focused-test.js'
13+
import noForceOption from './rules/no-force-option.js'
14+
import noGetByTitle from './rules/no-get-by-title.js'
15+
import noHooks from './rules/no-hooks.js'
16+
import noNestedStep from './rules/no-nested-step.js'
17+
import noNetworkidle from './rules/no-networkidle.js'
18+
import noNthMethods from './rules/no-nth-methods.js'
19+
import noPagePause from './rules/no-page-pause.js'
20+
import noRawLocators from './rules/no-raw-locators.js'
21+
import noRestrictedMatchers from './rules/no-restricted-matchers.js'
22+
import noSkippedTest from './rules/no-skipped-test.js'
23+
import noStandaloneExpect from './rules/no-standalone-expect.js'
24+
import noUnsafeReferences from './rules/no-unsafe-references.js'
25+
import noUselessAwait from './rules/no-useless-await.js'
26+
import noUselessNot from './rules/no-useless-not.js'
27+
import noWaitForSelector from './rules/no-wait-for-selector.js'
28+
import noWaitForTimeout from './rules/no-wait-for-timeout.js'
29+
import preferComparisonMatcher from './rules/prefer-comparison-matcher.js'
30+
import preferEqualityMatcher from './rules/prefer-equality-matcher.js'
31+
import preferHooksInOrder from './rules/prefer-hooks-in-order.js'
32+
import preferHooksOnTop from './rules/prefer-hooks-on-top.js'
33+
import preferLocator from './rules/prefer-locator.js'
34+
import preferLowercaseTitle from './rules/prefer-lowercase-title.js'
35+
import preferNativeLocators from './rules/prefer-native-locators.js'
36+
import preferStrictEqual from './rules/prefer-strict-equal.js'
37+
import preferToBe from './rules/prefer-to-be.js'
38+
import preferToContain from './rules/prefer-to-contain.js'
39+
import preferToHaveCount from './rules/prefer-to-have-count.js'
40+
import preferToHaveLength from './rules/prefer-to-have-length.js'
41+
import preferWebFirstAssertions from './rules/prefer-web-first-assertions.js'
42+
import requireHook from './rules/require-hook.js'
43+
import requireSoftAssertions from './rules/require-soft-assertions.js'
44+
import requireToThrowMessage from './rules/require-to-throw-message.js'
45+
import requireTopLevelDescribe from './rules/require-top-level-describe.js'
46+
import validDescribeCallback from './rules/valid-describe-callback.js'
47+
import validExpect from './rules/valid-expect.js'
48+
import validExpectInPromise from './rules/valid-expect-in-promise.js'
49+
import validTitle from './rules/valid-title.js'
5050

5151
const index = {
5252
configs: {},
@@ -150,6 +150,8 @@ const flatConfig = {
150150
},
151151
}
152152

153+
// @ts-expect-error We author this plugin in ESM, but export as CJS for
154+
// compatibility with ESLint<9. Long term, this will be changed to `export default`.
153155
export = {
154156
...index,
155157
configs: {

‎src/rules/expect-expect.test.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
import rule from '../../src/rules/expect-expect'
2-
import { javascript, runRuleTester } from '../utils/rule-tester'
1+
import rule from '../../src/rules/expect-expect.js'
2+
import { javascript, runRuleTester } from '../utils/rule-tester.js'
33

44
runRuleTester('expect-expect', rule, {
55
invalid: [

‎src/rules/expect-expect.ts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import ESTree from 'estree'
2-
import { dig } from '../utils/ast'
3-
import { createRule } from '../utils/createRule'
4-
import { parseFnCall } from '../utils/parseFnCall'
2+
import { dig } from '../utils/ast.js'
3+
import { createRule } from '../utils/createRule.js'
4+
import { parseFnCall } from '../utils/parseFnCall.js'
55

66
export default createRule({
77
create(context) {

‎src/rules/max-expects.test.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
import rule from '../../src/rules/max-expects'
2-
import { javascript, runRuleTester } from '../utils/rule-tester'
1+
import rule from '../../src/rules/max-expects.js'
2+
import { javascript, runRuleTester } from '../utils/rule-tester.js'
33

44
runRuleTester('max-expects', rule, {
55
invalid: [

0 commit comments

Comments
 (0)