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

ci(linting): fix typescript parser issue #11243

Merged
merged 1 commit into from
Jan 9, 2025
Merged

Conversation

benelan
Copy link
Member

@benelan benelan commented Jan 9, 2025

Summary

Resolve a lint-staged error reported by @macandcheese:

✖ eslint --fix:

/Users/adam9519/Documents/git-repositories/calcite-design-system/packages/calcite-components/src/components/combobox/combobox.stories.ts
  0:0  error  Parsing error: /Users/adam9519/Documents/git-repositories/calcite-design-system/packages/calcite-components/src/components/combobox/combobox.stories.ts was not found by the project service. Consider either including it in the tsconfig.json or including it in allowDefaultProject

✖ 1 problem (1 error, 0 warnings)

husky - pre-commit script failed (code 1)
Hook process exited.

It seems like the projectService option is using tsconfig.json (which excludes stories) instead of tsconfig-eslint.json. Disabling the option for now.

ref: https://typescript-eslint.io/getting-started/typed-linting/

@benelan benelan requested a review from alisonailea as a code owner January 9, 2025 20:49
@github-actions github-actions bot added the chore Issues with changes that don't modify src or test files. label Jan 9, 2025
@benelan benelan added the skip visual snapshots Pull requests that do not need visual regression testing. label Jan 9, 2025
@benelan
Copy link
Member Author

benelan commented Jan 9, 2025

pushing this through so stories can be committed, cc @jcfranco

@benelan benelan merged commit 2cdc294 into dev Jan 9, 2025
15 of 16 checks passed
@benelan benelan deleted the benelan/fix-tseslint-issue branch January 9, 2025 20:51
@benelan benelan mentioned this pull request Jan 21, 2025
benelan added a commit that referenced this pull request Jan 21, 2025
**Related Issue:** #11243 #11312

## Summary

Fix a linting error that was likely committed before the author's branch
was synced with the rule changes linked above. The linting error caused
issues committing during merge conflict resolution.
benelan added a commit that referenced this pull request Feb 8, 2025
## Summary

Resolve a lint-staged error reported by @macandcheese:

```text
✖ eslint --fix:

/Users/adam9519/Documents/git-repositories/calcite-design-system/packages/calcite-components/src/components/combobox/combobox.stories.ts
  0:0  error  Parsing error: /Users/adam9519/Documents/git-repositories/calcite-design-system/packages/calcite-components/src/components/combobox/combobox.stories.ts was not found by the project service. Consider either including it in the tsconfig.json or including it in allowDefaultProject

✖ 1 problem (1 error, 0 warnings)

husky - pre-commit script failed (code 1)
Hook process exited.
```

It seems like the projectService option is using `tsconfig.json` (which
excludes stories) instead of `tsconfig-eslint.json`. Disabling the
option for now.


ref: https://typescript-eslint.io/getting-started/typed-linting/
benelan added a commit that referenced this pull request Feb 8, 2025
**Related Issue:** #11243 #11312

## Summary

Fix a linting error that was likely committed before the author's branch
was synced with the rule changes linked above. The linting error caused
issues committing during merge conflict resolution.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Issues with changes that don't modify src or test files. skip visual snapshots Pull requests that do not need visual regression testing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant