MonkeyCanCode commented on PR #4953: URL: https://github.com/apache/polaris/pull/4953#issuecomment-5014636438
> > Hi @tarunjangid , thanks for the PR. I am not sure if this really close #4090 as all of this doing is just adding color to the output console. This doesn't necessary make it make much difference IMO. > > Thanks for the review — that’s a fair point. > > I agree this PR alone doesn’t fully address #4090. My intent was a small first step on `setup export` to validate direction before tackling the larger `setup apply` / dry-run UX work shown in the issue. > > If you’re aligned, I can extend this PR (or follow up immediately) with structured Rich output for export — e.g. a summary table + hierarchy tree on stderr while keeping plain YAML on stdout for redirection — and then do `setup apply --dry-run` in a separate PR, which seems like the main UX pain from #4090. > > Happy to follow whichever scope you prefer: expand this PR, or close/merge as a small incremental change and continue on apply/dry-run next. Hi @tarunjangid, Somehow I missed this response and sorry for the late response. I don't think it it worth atm to add a new library just to add color around yaml output. Thanks, Yong Zheng -- 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]
