Skip to content

Conversation

Netail
Copy link
Contributor

@Netail Netail commented Aug 21, 2025

Summary

Support incoming message & server response generics

Fixes # #19

@Netail Netail marked this pull request as draft August 21, 2025 22:11
@Netail Netail force-pushed the feat/typescript-generics branch 2 times, most recently from 63af615 to c38427e Compare August 21, 2025 23:23
@williamstein
Copy link
Contributor

Thanks! -- it seems to be failing tests.

@Netail
Copy link
Contributor Author

Netail commented Aug 22, 2025

Thanks! -- it seems to be failing tests.

Yup, a few things are left to fix. But off to bed, hence the draft :)

@williamstein
Copy link
Contributor

Thanks again!

@Netail Netail force-pushed the feat/typescript-generics branch 3 times, most recently from 08151fd to 84566ce Compare August 22, 2025 21:06
@Netail Netail force-pushed the feat/typescript-generics branch 5 times, most recently from d7e427e to b43b2af Compare August 30, 2025 00:07
@Netail Netail force-pushed the feat/typescript-generics branch from b43b2af to 5659027 Compare August 30, 2025 00:09
@Netail
Copy link
Contributor Author

Netail commented Aug 30, 2025

@williamstein PR should be ready, but the migration from jest to vitest broke some cicd

@Netail Netail marked this pull request as ready for review August 30, 2025 00:13
@williamstein
Copy link
Contributor

I'll fix the CI/CD now.

@williamstein williamstein merged commit 3d0821f into sagemathinc:main Aug 30, 2025
0 of 6 checks passed
@williamstein
Copy link
Contributor

This looks good to me, passes now, and is [email protected].

@Netail Netail deleted the feat/typescript-generics branch August 30, 2025 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants