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

Split all providers into "download" and "process" steps #26

Open
wagoodman opened this issue Dec 21, 2022 · 0 comments
Open

Split all providers into "download" and "process" steps #26

wagoodman opened this issue Dec 21, 2022 · 0 comments
Labels
enhancement New feature or request refactor highlights the need for a refactor

Comments

@wagoodman
Copy link
Contributor

Most of the parsers internally have a _download and _process (or similar) methods to organize the work, however, there is no strict separation of these processes when calling the get method on the parsers. Ideally there should be a way to download all state in one step and process the state in another. This could be exposed on the CLI as well (e.g. vunnel download <provider> vunnel process <provider> and leave vunnel run provider as is, calling both download and process internally).

This would help with provider development, where most of the development is on the processing-side, so naturally allowing for download to be separate and a guarantee of no network calls / updating of input state when calling "process".

@wagoodman wagoodman added the enhancement New feature or request label Dec 21, 2022
@wagoodman wagoodman added the refactor highlights the need for a refactor label Jan 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request refactor highlights the need for a refactor
Projects
None yet
Development

No branches or pull requests

1 participant