Skip to content

refactor: Rename firebase client type#622

Merged
erezrokah merged 1 commit into
cloudquery:mainfrom
erezrokah:refactor/rename_firebase_client
May 3, 2022
Merged

refactor: Rename firebase client type#622
erezrokah merged 1 commit into
cloudquery:mainfrom
erezrokah:refactor/rename_firebase_client

Conversation

@erezrokah
Copy link
Copy Markdown
Member

Follow up on #618

@erezrokah erezrokah requested review from a team, disq and irmatov and removed request for a team and irmatov May 2, 2022 08:39
Copy link
Copy Markdown
Contributor

@roneli roneli left a comment

Choose a reason for hiding this comment

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

Move this under registry, moreover merge this after #623 is merged.

@erezrokah erezrokah force-pushed the refactor/rename_firebase_client branch from 5ff6e8b to f853182 Compare May 3, 2022 11:57
@erezrokah erezrokah requested a review from roneli May 3, 2022 11:57
@erezrokah
Copy link
Copy Markdown
Member Author

@roneli rebased the PR

Copy link
Copy Markdown
Contributor

@roneli roneli left a comment

Choose a reason for hiding this comment

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

I think this should maybe move to hub package as its only related to it, maybe in a separate PR.

@erezrokah erezrokah merged commit 09e3609 into cloudquery:main May 3, 2022
@erezrokah erezrokah deleted the refactor/rename_firebase_client branch May 3, 2022 14:07
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.

3 participants