-
-
Notifications
You must be signed in to change notification settings - Fork 696
feat: introduce tsdown
, support mixed js & ts in codebase
#2916
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Changes from 10 commits
b9ad774
928fe26
1b24273
0f786d8
aedcd8d
bd930b7
79d440d
18d7488
861c1d6
3d7f91c
f1c5923
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -14,3 +14,4 @@ yarn-error.log | |
/docs/.vitepress/cache | ||
typings/eslint/lib/rules | ||
eslint-typegen.d.ts | ||
dist |
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
import { name, version } from '../package.json' with { type: 'json' } | ||
|
||
export { name, version } |
Original file line number | Diff line number | Diff line change | ||||||
---|---|---|---|---|---|---|---|---|
|
@@ -2,8 +2,8 @@ | |||||||
"name": "eslint-plugin-vue", | ||||||||
"version": "10.4.0", | ||||||||
"description": "Official ESLint plugin for Vue.js", | ||||||||
"main": "lib/index.js", | ||||||||
"types": "lib/index.d.ts", | ||||||||
"main": "dist/index.js", | ||||||||
"types": "dist/index.d.ts", | ||||||||
"scripts": { | ||||||||
"new": "node tools/new-rule.js", | ||||||||
"start": "npm run test:base -- --watch --growl", | ||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm not a maintainer, just a quick note 🙇 CI seems to fail because dist isn’t built before tests.
Suggested change
Thanks for the great work 🚀 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks for your reminder, but this is not the only issue. I am currently waiting for the rolldown team to resolve the cjs-related issue: rolldown/rolldown#5930. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks for the heads-up 👍 |
||||||||
|
@@ -22,15 +22,17 @@ | |||||||
"update": "node ./tools/update.js", | ||||||||
"update-resources": "node ./tools/update-resources.js", | ||||||||
"typegen": "node ./tools/generate-typegen.mjs", | ||||||||
"build": "tsdown", | ||||||||
"docs:watch": "vitepress dev docs", | ||||||||
"predocs:build": "npm run update", | ||||||||
"docs:build": "vitepress build docs", | ||||||||
"generate:version": "env-cmd -e version npm run update && npm run lint -- --fix", | ||||||||
"changeset:version": "changeset version && npm run generate:version && git add --all", | ||||||||
"changeset:publish": "npm run typegen && changeset publish" | ||||||||
"changeset:publish": "npm run typegen && changeset publish", | ||||||||
"prepublishOnly": "npm run build" | ||||||||
}, | ||||||||
"files": [ | ||||||||
"lib" | ||||||||
"dist" | ||||||||
], | ||||||||
"homepage": "https://eslint.vuejs.org", | ||||||||
"keywords": [ | ||||||||
|
@@ -115,6 +117,7 @@ | |||||||
"markdownlint-cli": "^0.42.0", | ||||||||
"pathe": "^1.1.2", | ||||||||
"prettier": "^3.3.3", | ||||||||
"tsdown": "^0.14.1", | ||||||||
"typescript": "^5.7.2", | ||||||||
"vite-plugin-eslint4b": "^0.5.1", | ||||||||
"vitepress": "^1.4.1", | ||||||||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
import { defineConfig } from 'tsdown' | ||
|
||
export default defineConfig({ | ||
target: 'node18', | ||
entry: ['lib/index.js'], | ||
format: ['cjs'], | ||
copy: ['lib/index.d.ts'], | ||
dts: false, | ||
external: ['typescriopt'], | ||
FloEdelmann marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
unbundle: true | ||
}) |
Uh oh!
There was an error while loading. Please reload this page.