Skip to content

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
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