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

clientgen/ts: qualify Request with globalThis to avoid name conflicts #1721

Merged
merged 1 commit into from
Jan 20, 2025

Conversation

fredr
Copy link
Member

@fredr fredr commented Jan 17, 2025

Qualify Response with globalThis to avoid name conflicts with types used within that service.

globalThis is available in all browsers and in nodejs

@fredr fredr requested a review from eandre January 17, 2025 14:17
@fredr fredr self-assigned this Jan 17, 2025
@encore-cla
Copy link

encore-cla bot commented Jan 17, 2025

All committers have signed the CLA.

@fredr fredr changed the title clientgen/ts: qualify the type name with globalThis to avoid name conflicts clientgen/ts: qualify Request with globalThis to avoid name conflicts Jan 17, 2025
@fredr fredr force-pushed the fredr/clientgen-ts-conflicting-types branch from e2bbf1f to 2d090d2 Compare January 20, 2025 07:28
@fredr fredr merged commit e0bff4b into main Jan 20, 2025
4 checks passed
@fredr fredr deleted the fredr/clientgen-ts-conflicting-types branch January 20, 2025 07:46
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