Skip to content

Conversation

@jgaehring
Copy link

These changes to the codegen repository corresponds to datafoodconsortium/connector-typescript#16, which contains the actual Connector library and distributables built with the Acceleo runner and the TypeScript compiler.

Note that I'm opening this on the 1.2.0 branch still, since main has yet to merge that release.

@jgaehring jgaehring changed the title TypeScript Observers TypeScript Observers for codegen May 13, 2025
@RaggedStaff RaggedStaff requested a review from lecoqlibre May 14, 2025 20:52
@RaggedStaff RaggedStaff moved this from Icebox ❄ to To review ☕ in Tech meeting board May 14, 2025
@jgaehring jgaehring marked this pull request as draft May 28, 2025 00:00
@jgaehring
Copy link
Author

@lecoqlibre: I realized this only includes the TypeScript implementation of the observers. Do we want to wait until the Ruby and PHP implementations are ready?

I converted it back to a draft for the meantime. The variant changes and submodule work (#30) seemed to me like they ought to be merged first, because the TypeScript Observer still seems more susceptible to change until the Ruby/PHP work is done. And since alpha.11 of connector-typescript is already published on NPM, there's no urgency to merge this one into connector-codegen.

@jgaehring jgaehring moved this from To review ☕ to In Progress in Tech meeting board May 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

1 participant