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

Refactor RemotingService and SyncFrom... interfaces #146

Merged
merged 14 commits into from
Jun 12, 2024

Conversation

githublaohu
Copy link
Contributor

Fixes #issue_id

Motivation

Explain the content here.
Explain why you want to make the changes and what problem you're trying to solve.

Modifications

Describe the modifications you've done.

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
  • If a feature is not applicable for documentation, explain why?
  • If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

@Pil0tXia
Copy link
Member

Pil0tXia commented Jun 2, 2024

@githublaohu Please fix CI errors.

image

Checkstyle guide: https://eventmesh.apache.org/zh/community/contribute/contribute/

@Pil0tXia
Copy link
Member

Pil0tXia commented Jun 2, 2024

Copy link
Member

@Pil0tXia Pil0tXia left a comment

Choose a reason for hiding this comment

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

Subsequent work of this PR:

Comment on lines 181 to 185
/**
* 解除玩关系,才能删除
*
* @param clusterEntityList
*/
Copy link
Member

Choose a reason for hiding this comment

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

typo

@Pil0tXia Pil0tXia changed the title fix(core):1. 重构 同步模块 2. 重构 remote 模块 Refactor RemotingService and SyncFrom... interfaces Jun 12, 2024
@Pil0tXia Pil0tXia merged commit d3ffcb5 into apache:main Jun 12, 2024
5 checks passed
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.

2 participants