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

feat: add types to package #1667

Merged
merged 1 commit into from
Nov 5, 2024
Merged

Conversation

mskelton
Copy link
Contributor

@mskelton mskelton commented Nov 4, 2024

Fixes #1469

I opted to generate the types manually with a hard coded index.d.ts file given that the types are very straightforward and the result works much better with typescript-eslint in my experience vs simply relying on tsc generated types.

Copy link
Collaborator

@G-Rath G-Rath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, a great start that just needs a few tweaks

index.d.ts Outdated Show resolved Hide resolved
index.d.ts Outdated Show resolved Hide resolved
environments: {
globals: {
globals: {
[key: string]: boolean;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note: ideally this should be Linter.GlobalConf, but that was introduced in @types/eslint@9

.eslintignore Outdated Show resolved Hide resolved
@mskelton
Copy link
Contributor Author

mskelton commented Nov 4, 2024

@G-Rath Updated per your suggestions

index.d.ts Outdated Show resolved Hide resolved
Copy link
Collaborator

@G-Rath G-Rath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome, thanks!

@G-Rath G-Rath merged commit 1ce1258 into jest-community:main Nov 5, 2024
44 checks passed
github-actions bot pushed a commit that referenced this pull request Nov 5, 2024
# [28.9.0](v28.8.3...v28.9.0) (2024-11-05)

### Features

* add TypeScript types ([#1667](#1667)) ([1ce1258](1ce1258))
@G-Rath
Copy link
Collaborator

G-Rath commented Nov 5, 2024

Apparently there's a recent issue with semantic-release that means it cannot find pull requests when looking to post its "this was released in ..." message, which should hopefully be fixed in the next few days (semantic-release/github#960) 😅

The release was still successful, so this has gone out in v28.9.0

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.

Feature: Generate typings for rules etc.
2 participants