Skip to content

Conversation

@bokket
Copy link
Contributor

@bokket bokket commented Jan 1, 2024

CC @Xuanwo

@bokket bokket requested a review from Xuanwo as a code owner January 1, 2024 15:01
@github-actions github-actions bot added the releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" label Jan 1, 2024
Signed-off-by: bokket <[email protected]>
@Xuanwo Xuanwo changed the title refactor(services/ftp):Don't impl From<XxxError> for Error refactor(services/ftp): Impl parse_error instead of From<Error> Jan 1, 2024
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Great, thanks!

@Xuanwo Xuanwo merged commit e925b4b into apache:main Jan 1, 2024
@Xuanwo Xuanwo mentioned this pull request Jan 1, 2024
5 tasks
@Xuanwo
Copy link
Member

Xuanwo commented Jan 1, 2024

CC @Xuanwo

Hi, it's preferable to link related issues rather than pinging me directly. This provides the reviewer with more information and allows them to participate in the review process as well.

Let's improve this part in next PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants