Skip to content

fix: Sync support optional provider args#642

Merged
roneli merged 1 commit into
cloudquery:mainfrom
roneli:fix/optional_provider_sync_commands
May 4, 2022
Merged

fix: Sync support optional provider args#642
roneli merged 1 commit into
cloudquery:mainfrom
roneli:fix/optional_provider_sync_commands

Conversation

@roneli
Copy link
Copy Markdown
Contributor

@roneli roneli commented May 3, 2022

fix support optional providers for sync commands

Closes #634

fix support optional providers for sync commands
@roneli roneli requested review from a team, bbernays and shimonp21 and removed request for a team May 3, 2022 14:27
@github-actions github-actions Bot added the fix label May 3, 2022
Copy link
Copy Markdown
Contributor

@shimonp21 shimonp21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :)

@roneli roneli requested a review from disq May 3, 2022 17:11
@roneli roneli merged commit 5eac023 into cloudquery:main May 4, 2022
@roneli roneli deleted the fix/optional_provider_sync_commands branch May 4, 2022 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

build-schema/upgrade/downgrade: Optional [provider] variable is not optional

3 participants