Skip to content

Update dependencies, improve tests, and modify build configuration#493

Merged
thednp merged 9 commits intothednp:masterfrom
LoicE5:various-fixes
Mar 17, 2026
Merged

Update dependencies, improve tests, and modify build configuration#493
thednp merged 9 commits intothednp:masterfrom
LoicE5:various-fixes

Conversation

@LoicE5
Copy link
Contributor

@LoicE5 LoicE5 commented Mar 16, 2026

Hey there 😄

Working with some packages having bootstrap.native as peer deps, I had some issues regarding vitest.config.ts file discovery, and I thought I could contribute. Issue is that these files are in the pushed code on npmjs. → https://www.npmjs.com/package/bootstrap.native?activeTab=code

In this PR :

  • Replaced .npmignore with a "files": ["dist"] field in package.json to ensure only dist, README, LICENSE, package.json are included in the published package on npm registry.

As well as :

  • Bump @thednp/event-listener to 2.0.12
  • Switch build minifier from esbuild to oxc (vite.config.ts, vite.config.components.ts) — esbuild is now deprecated in Vite
  • Add deno.json with sloppy-imports lint rule (was blocking test runs)
  • Switch browser target in test-ui from chrome to chromium
  • Improve scrollspy test stability (delays + viewport)
  • Update README dependency version badges
  • Add .pnpm-store to .gitignore

The rebuilt dist/ output reflecting the oxc minifier change is carried in a separate PR → #494.

Hope it helps! 😀

@LoicE5 LoicE5 mentioned this pull request Mar 16, 2026
@thednp thednp marked this pull request as ready for review March 17, 2026 06:22
@thednp thednp merged commit c5f2f80 into thednp:master Mar 17, 2026
3 of 4 checks passed
@thednp
Copy link
Owner

thednp commented Mar 17, 2026

Thank you for the contribution.

I will recheck everything and hopefully, later today, we can publish v5.1.7.

@thednp
Copy link
Owner

thednp commented Mar 17, 2026

@LoicE5 please do a test with v5.1.7 and report back any issues.

@LoicE5
Copy link
Contributor Author

LoicE5 commented Mar 17, 2026

Just tested it now and it works like a charm. Thanks 🙌🏻 @thednp

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