Skip to content

fix: lexer.d.mts was never published - #119

Merged
guybedford merged 1 commit into
nodejs:mainfrom
JounQin:patch-1
Aug 10, 2026
Merged

fix: lexer.d.mts was never published#119
guybedford merged 1 commit into
nodejs:mainfrom
JounQin:patch-1

Conversation

@JounQin

@JounQin JounQin commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

related #85

cc @guybedford

Copilot AI lite review requested due to automatic review settings August 10, 2026 08:15

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Ensures the lexer.d.mts TypeScript declaration file is actually included in the published npm package, aligning the published artifacts with the existing exports.import.types entry.

Changes:

  • Add lexer.d.mts to the files whitelist in package.json so it’s included in the npm tarball.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@guybedford
guybedford merged commit 020eeee into nodejs:main Aug 10, 2026
1 check passed
@JounQin
JounQin deleted the patch-1 branch August 10, 2026 18:31
@JounQin

JounQin commented Aug 11, 2026

Copy link
Copy Markdown
Contributor Author

@guybedford Would you cut a new release for this fix?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants