Skip to content

refactor(types): migrate from tsd to tstyche#43

Merged
Tony133 merged 5 commits intomainfrom
refactor/types-migration-tstyche
May 7, 2026
Merged

refactor(types): migrate from tsd to tstyche#43
Tony133 merged 5 commits intomainfrom
refactor/types-migration-tstyche

Conversation

@Tony133
Copy link
Copy Markdown
Member

@Tony133 Tony133 commented May 6, 2026

Proposals:

  • migrate from tsd to tstyche

  • GlideClient.createClient with invalid options does not properly reject on macOS, causing fastify to hit the plugin timeout instead of catching the error. ( continuous integration )

    • Add port: 9999 to the invalid host address in the connection test
    • Add pluginTimeout: 20000 to stabilize tests that involve connection errors
    • Skip both "should throw" tests on darwin platform as a temporary workaround due to GlideClient.createClient not properly rejecting on macOS with invalid options ( continuous integration )

PR of Reference:

@Tony133 Tony133 marked this pull request as ready for review May 6, 2026 07:07
@Tony133 Tony133 marked this pull request as draft May 6, 2026 07:14
@Tony133 Tony133 force-pushed the refactor/types-migration-tstyche branch 3 times, most recently from aadba72 to 756c838 Compare May 6, 2026 07:43
@Tony133 Tony133 force-pushed the refactor/types-migration-tstyche branch from 7d96710 to aeb2f24 Compare May 6, 2026 07:49
@Tony133 Tony133 force-pushed the refactor/types-migration-tstyche branch from 5297b62 to 36aeefe Compare May 6, 2026 08:02
@Tony133 Tony133 marked this pull request as ready for review May 6, 2026 08:15
Comment thread types/index.tst.ts Outdated
Comment thread types/index.tst.ts Outdated
@Tony133 Tony133 requested a review from mrazauskas May 6, 2026 20:56
@mrazauskas
Copy link
Copy Markdown

lgtm

@Tony133 Tony133 merged commit 6876066 into main May 7, 2026
12 checks passed
@Tony133 Tony133 deleted the refactor/types-migration-tstyche branch May 7, 2026 06:48
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.

2 participants