Skip to content

Conversation

@mabdh
Copy link
Member

@mabdh mabdh commented Mar 24, 2022

No description provided.

@ravisuhag
Copy link
Member

@mabdh can you please check the lint issues reported by buf on compass proto.

@mabdh
Copy link
Member Author

mabdh commented Mar 24, 2022

@ravisuhag yes there are some breaking changes with the main, how could we force the changes for compass proto?
is it fine to merge it later with these breaking changes?

@ravisuhag
Copy link
Member

@ravisuhag yes there are some breaking changes with the main, how could we force the changes for compass proto? is it fine to merge it later with these breaking changes?

It is fine to merge with these changes since we are yet to integrate and release the apis. But there are some more lint issues, which also we can fix.

@mabdh
Copy link
Member Author

mabdh commented Mar 24, 2022

@ravisuhag right, there are some lint errors because we re-use a message in the different rpc services, e.g. google.protobuf.Empty.
Sure I will fix it.

@mabdh mabdh self-assigned this Mar 24, 2022
@mabdh mabdh force-pushed the compass-assets-domain branch 6 times, most recently from 8a213c7 to c3146a2 Compare March 29, 2022 07:39
@mabdh mabdh marked this pull request as ready for review March 29, 2022 07:45
@mabdh mabdh force-pushed the compass-assets-domain branch from c3146a2 to c8ff339 Compare March 31, 2022 02:53
@mabdh mabdh changed the title feat(compass): add asset, search, lineage handlers feat(compass): add asset, search, lineage, users, star, tags handlers Mar 31, 2022
@mabdh mabdh force-pushed the compass-assets-domain branch from c8ff339 to d2e6328 Compare March 31, 2022 03:54
@mabdh mabdh force-pushed the compass-assets-domain branch from d2e6328 to 7b6fd2e Compare March 31, 2022 03:56
@mabdh mabdh merged commit 4c76e08 into main Mar 31, 2022
@mabdh mabdh deleted the compass-assets-domain branch March 31, 2022 03:58
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