Skip to content

feat: add flow-ts package#22087

Closed
Lodin wants to merge 3 commits into
mainfrom
feat/client-code
Closed

feat: add flow-ts package#22087
Lodin wants to merge 3 commits into
mainfrom
feat/client-code

Conversation

@Lodin
Copy link
Copy Markdown
Contributor

@Lodin Lodin commented Aug 24, 2025

This PR adds an npm package that contains all TS/JS code used by Flow

@tepi tepi marked this pull request as ready for review August 27, 2025 08:33
@tepi tepi marked this pull request as draft August 27, 2025 08:33
@sonarqubecloud
Copy link
Copy Markdown

@mshabarov
Copy link
Copy Markdown
Contributor

To be tested against workflow/usability:

pros are having proper JS/TS dev environment and good preparation for further merge of repos (if decided),

cons may be the more complex debugging/testing process for Flow developers as then they would need to additionally build npm package, link it to a testing project, also the CI/CD pipelines would need significant changes.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Oct 22, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ tepi
❌ Lodin
You have signed the CLA already but the status is still pending? Let us recheck it.

@mshabarov
Copy link
Copy Markdown
Contributor

Let's create a ticket related to this and estimate effort needed to compile this refactoring and make changes to CI/CD against value we get after this.

@mshabarov
Copy link
Copy Markdown
Contributor

Ticket created #23210.

@mshabarov mshabarov closed this Jan 14, 2026
@github-project-automation github-project-automation Bot moved this from 🔎Iteration reviews to Done in Vaadin Flow | Hilla | Kits ongoing work Jan 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

5 participants