Skip to content

Comments

fix: fixed missing types declarations in packages#229

Merged
ValDesign22 merged 3 commits intomasterfrom
fix/missing-types-declarations
Jan 27, 2026
Merged

fix: fixed missing types declarations in packages#229
ValDesign22 merged 3 commits intomasterfrom
fix/missing-types-declarations

Conversation

@ValDesign22
Copy link
Member

This pull request updates the packaging configuration for all packages in the monorepo to ensure that only the necessary build artifacts are published to npm. The changes mainly focus on improving the .npmignore files and updating package.json settings to specify the correct files and type declarations for npm distribution.

Packaging improvements:

  • Added the files field with ["dist"] and the types field pointing to dist/index.d.ts in all package.json files to ensure only compiled code and type definitions are published. [1] [2] [3] [4] [5]

NPM ignore file updates:

  • Removed the top-level .npmignore files from all packages (including core, discordjs, discordjs-light, eris, and oceanic) to prevent source files, tests, and configuration files from being included in npm packages.
  • Updated packages/core/.npmignore to rename the test directory to tests and added .zed to the ignore list for more accurate exclusion.

@ValDesign22 ValDesign22 self-assigned this Jan 27, 2026
@ValDesign22 ValDesign22 added the bug Something isn't working label Jan 27, 2026
@ValDesign22
Copy link
Member Author

checks for node 18.x needs to be removed since the test packages aren't compatible with it anymore

@ValDesign22 ValDesign22 merged commit e26a649 into master Jan 27, 2026
3 of 5 checks passed
@ValDesign22 ValDesign22 deleted the fix/missing-types-declarations branch January 27, 2026 13:55
@github-actions
Copy link

🎉 This PR is included in version 2.8.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

bug Something isn't working released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants