Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CLI: Support state-props-to-remove? #591

Open
josephjclark opened this issue Feb 8, 2024 · 0 comments
Open

CLI: Support state-props-to-remove? #591

josephjclark opened this issue Feb 8, 2024 · 0 comments

Comments

@josephjclark
Copy link
Collaborator

josephjclark commented Feb 8, 2024

We should probably think about whether and how we include this in the CLI.

When the CLI runs locally it may not be necessary or desirable to remove any props from the returned state. All config should be known to the user anyway (for now - if we integrate a credential loader with Lightning then this is no longer true).

But the CLI and worker should have feature parity and have similar behaviour. So I do think we need to include this flag with the same default value as the worker, if only for posterity.

It's a small feature but I'm not interested in adding it to the 1.0 release queue.

@github-project-automation github-project-automation bot moved this to New Issues in v2 Feb 8, 2024
@christad92 christad92 moved this from New Issues to Icebox in v2 Feb 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Icebox
Development

No branches or pull requests

1 participant