Skip to content
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

Run internal samples with skipLibCheck: false? #10826

Open
maxpatiiuk opened this issue Nov 21, 2024 · 0 comments
Open

Run internal samples with skipLibCheck: false? #10826

maxpatiiuk opened this issue Nov 21, 2024 · 0 comments
Labels
0 - new New issues that need assignment. needs milestone Planning workflow - pending milestone assignment, has priority and/or estimate. p - low Issue is non core or affecting less that 10% of people using the library testing Issues related to automated or manual testing.

Comments

@maxpatiiuk
Copy link
Member

Priority impact

p - low

Test type

Integration

Which Component(s)

all

Unstable Tests

No response

Test error, if applicable

Calcite's samples at the moment appear to set the skipLibCheck: true option:

https://github.com/Esri/calcite-design-system/blob/main/examples/components/react/tsconfig.app.json#L7

This option disables type-checking of Calcite's output .d.ts files. While the option is commonly recommended, it is not on by default in TypeScript, not on by default in many starter templates out there, and can not be enabled in some projects (like JS API).

Thus, it might be advisable for Calcite's internal samples testing to also have this option disabled, to opt into additional checks.

This check will detect issues like the following earlier in the future, before they affect consumers:

PR skipped, if applicable

No response

Additional Info

No response

@maxpatiiuk maxpatiiuk added testing Issues related to automated or manual testing. 0 - new New issues that need assignment. needs milestone Planning workflow - pending milestone assignment, has priority and/or estimate. labels Nov 21, 2024
@github-actions github-actions bot added the p - low Issue is non core or affecting less that 10% of people using the library label Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 - new New issues that need assignment. needs milestone Planning workflow - pending milestone assignment, has priority and/or estimate. p - low Issue is non core or affecting less that 10% of people using the library testing Issues related to automated or manual testing.
Projects
None yet
Development

No branches or pull requests

1 participant