Vamsi-klu commented on PR #67054: URL: https://github.com/apache/airflow/pull/67054#issuecomment-4490455377
Thanks, I addressed the requested changes. Updated the param store helpers to use the simpler `hydrateParams` / `serializeConf` naming, removed the extra JSON parse helper, and now parse inline where needed. I also simplified the hydration/serialization logic, kept `description: param.description`, and updated the tests to use `JSON.parse(...)` directly. I rebased onto current `main` and updated `FieldMultiType` so valid changes call `setParamsDict(paramsDict, name)`, while invalid input still preserves the previous valid value. Validation run locally: - `corepack [email protected] --config.verify-deps-before-run=false test useParamStore FieldMultiType ConnectionForm` - `corepack [email protected] --config.verify-deps-before-run=false lint` - `prek run --from-ref upstream/main --stage pre-commit` --- Drafted-by: Codex (GPT-5); reviewed by @Vamsi-klu before posting -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
