Skip to content

fix: correct generated typescript types#559

Merged
fatso83 merged 1 commit intosinonjs:mainfrom
AviVahl:fix-generated-types
Apr 10, 2026
Merged

fix: correct generated typescript types#559
fatso83 merged 1 commit intosinonjs:mainfrom
AviVahl:fix-generated-types

Conversation

@AviVahl
Copy link
Copy Markdown
Contributor

@AviVahl AviVahl commented Apr 3, 2026

Purpose (TL;DR) - mandatory

fixes #558

ensure generated .d.ts has no duplicate withGlobal definitions, where one is exported and the other isn't.

regenerated types using npm run types:build

Solution - optional

explicitly set the type of the withGlobal export

ensure generated .d.ts has no duplicate withGlobal definitions, where one is exported and the other isn't.

regenerated types using `npm run types:build`
@AviVahl AviVahl force-pushed the fix-generated-types branch from 3e1e6fd to 19478b5 Compare April 8, 2026 21:08
@AviVahl
Copy link
Copy Markdown
Contributor Author

AviVahl commented Apr 8, 2026

I've rebased to latest main.

@fatso83 any chance you can take a look? it's been a week since the break and everyone are locking to older versions because of this.

@fatso83
Copy link
Copy Markdown
Contributor

fatso83 commented Apr 10, 2026

Hi, I expected some fallout from this, but for whatever reason, I did not see any notifications until today 🤷 Looking at it now.

Copy link
Copy Markdown
Contributor

@fatso83 fatso83 left a comment

Choose a reason for hiding this comment

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

Thanks!

@fatso83 fatso83 merged commit a136d80 into sinonjs:main Apr 10, 2026
11 checks passed
@AviVahl AviVahl deleted the fix-generated-types branch April 10, 2026 19:54
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.

TypeScript doesn't compile

2 participants