Add config for running web-forms via containers#327
Conversation
|
4f6ea80 to
defd256
Compare
|
Thanks so much for sharing this and I'm sorry we didn't give you feedback before now! I'm going to close this because we're in the process of significantly reworking how Web Form's code is organized. In particular, we've spun https://github.com/getodk/tree-sitter-xpath out as a separate repo so that building the Web Forms packages on top of it don't require rebuilding the native dependency. We don't expect that package to change much. We will be moving the rest of the packages into the Central Frontend repo shortly. We'll continue publishing them as we do now but will develop them in the frontend monorepo so that we can more easily have end-to-end testing with Central which is our true integration target. We'll spin the preview out into the website repository. We'll post more details about all of this on the forum shortly but wanted to give you some context here first. |
I have verified this PR works in these browsers (latest versions):
What else has been done to verify that this works as intended?
Why is this the best possible solution? Were any other approaches considered?
How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?
No change.
Do we need any specific form for testing your changes? If so, please attach one.
See above.
What's changed
What I included:
contrib/containers, suggesting it's not maintained as part of the core library (not officially maintained / guaranteed to work). I'm not sure if everyone is as keen on containers as I am, so thought this most appropriate.Alternatives:
yarnused as normal for local development.Open to suggestions of alternative approaches, or feel free to close this PR if it's not useful 😄
Update 15-04-2025: