Skip to content

changed rollup-plugin-ts to @rollup/plugin-typescript #151

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Nimishxyz
Copy link

This pull request includes changes to the build configuration and dependencies to solve this #issue #140

Build configuration updates:

  • build.mjs: Replaced rollup-plugin-ts with @rollup/plugin-typescript for better TypeScript integration. Updated the tsconfig property to compilerOptions and added declarationMap to the configuration. [1] [2]
  • build.mjs: Changed the output file path to use joinPath for consistency and to ensure the correct output directory is used.

Dependency updates:

  • package.json: Added @rollup/plugin-typescript as a development dependency and removed rollup-plugin-ts. [1] [2]

there is still one issue where build process is not completing and getting stuck in the end.

@Nimishxyz
Copy link
Author

@shalithasuranga, could you please review the issue and provide a solution if possible?

@shalithasuranga
Copy link
Member

Hello @Nimishxyz. Thanks so much for sending this pull request. We can merge this once you solve the build script issue. Thanks 🎉

@Nimishxyz
Copy link
Author

@shalithasuranga I have fixed the build script issue and you can check and merged once satisfied.

@shalithasuranga
Copy link
Member

Hello @Nimishxyz, thanks for your update. We can merge this once if you solve the remaining error messages and properly generate TypeScript files similar to the current main branch version. Thanks 🎉

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