Skip to content

Conversation

skyfly18
Copy link
Contributor

@skyfly18 skyfly18 commented Aug 6, 2025

fixed #1201

@f3l1x
Copy link
Member

f3l1x commented Aug 6, 2025

Why do you need phpstan config and others? I thought you missing TS type definition (*.d.ts) files.

@skyfly18
Copy link
Contributor Author

skyfly18 commented Aug 6, 2025

I don't need phpstan config and ruleset.xml

@f3l1x
Copy link
Member

f3l1x commented Aug 6, 2025

Why do you need rollup.config.js and others? It's only for this repository. I don't see the point. Tell me your motivation.

@skyfly18
Copy link
Contributor Author

skyfly18 commented Aug 6, 2025

package.json because Dependency Management.

@f3l1x
Copy link
Member

f3l1x commented Aug 6, 2025

And others?

@skyfly18
Copy link
Contributor Author

skyfly18 commented Aug 8, 2025

To create the Datagrid JavaScript files, we use npm install %localpath%. For this, we use the tsconfig and rollup.config files. Therefore, it would be helpful if the three files package.json, tsconfig.json and rollup.config.js remain in the export.

@f3l1x
Copy link
Member

f3l1x commented Aug 11, 2025

Why do you need these files?

rollup.config.js export-ignore
tsconfig.json export-ignore

@radimvaculik
Copy link
Member

@skyfly18 ping.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Package without package.json and typescript config

3 participants