Skip to content

chore: update pattern to ONLY include top-level js files in dist#820

Merged
faisalman merged 1 commit into
faisalman:masterfrom
hyperz111:end-user-ignore
Jan 13, 2026
Merged

chore: update pattern to ONLY include top-level js files in dist#820
faisalman merged 1 commit into
faisalman:masterfrom
hyperz111:end-user-ignore

Conversation

@hyperz111

@hyperz111 hyperz111 commented Jan 12, 2026

Copy link
Copy Markdown
Contributor

Prerequisites

Type of Change

chore

Description

Update pattern in files field to ONLY include top-level JavaScript files in dist directory when publishing.
The JavaScript files are:

  • dist/ua-parser.min.js
  • dist/ua-parser.min.mjs
  • dist/ua-parser.pack.js.
  • dist/ua-parser.pack.mjs

Don't worry, this change is ONLY affected to dist directory.

Test

Nothing

Impact

Nothing

Other Info

You can run npm pack to see differents.

Before:

npm notice package size: 771.9 kB
npm notice unpacked size: 1.3 MB

After:

npm notice package size: 150.0 kB
npm notice unpacked size: 477.8 kB

@faisalman faisalman left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Hi, thanks for contributing to the ua-parser-js project, before I can merge this, can you please check the CLA in the submission form?

@hyperz111

Copy link
Copy Markdown
Contributor Author

Hi, thanks for contributing to the ua-parser-js project, before I can merge this, can you please check the CLA in the submission form?

I have check it. Can you merge it?

@faisalman
faisalman merged commit f1eed9e into faisalman:master Jan 13, 2026
6 checks passed
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.

2 participants