Skip to content

Add elems into FieldType#208

Closed
Xuanwo wants to merge 9 commits into
pingcap:masterfrom
Xuanwo:patch-1
Closed

Add elems into FieldType#208
Xuanwo wants to merge 9 commits into
pingcap:masterfrom
Xuanwo:patch-1

Conversation

@Xuanwo
Copy link
Copy Markdown
Contributor

@Xuanwo Xuanwo commented Feb 1, 2021

No description provided.

@Xuanwo
Copy link
Copy Markdown
Contributor Author

Xuanwo commented Feb 1, 2021

The previous change is for testing CI, please ignore.

Copy link
Copy Markdown
Contributor

@wshwsh12 wshwsh12 left a comment

Choose a reason for hiding this comment

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

LGTM

@dyzsr
Copy link
Copy Markdown

dyzsr commented Mar 4, 2021

@Xuanwo can you resolve the conflicts?

@Xuanwo
Copy link
Copy Markdown
Contributor Author

Xuanwo commented Mar 4, 2021

OK

@Xuanwo
Copy link
Copy Markdown
Contributor Author

Xuanwo commented Mar 4, 2021

Fixed

@dyzsr
Copy link
Copy Markdown

dyzsr commented Mar 4, 2021

Thanks

@Xuanwo
Copy link
Copy Markdown
Contributor Author

Xuanwo commented Mar 4, 2021

So we got to merge this PR?

@windtalker
Copy link
Copy Markdown
Contributor

I think we can close is pr since #217 did the same thing, and has been merged.

@Xuanwo
Copy link
Copy Markdown
Contributor Author

Xuanwo commented Mar 26, 2021

I had to close my PR after someone else repeated my work.

Did I do something wrong? I don't think so.

This PR was created in February and the conflict was handled 22 days ago with the review request, but a PR created 17 days ago has already been reviewed and merged.

If there were no plans to merge my PR, please let me know in advance. I sincerely hope this doesn't happen again.

@Xuanwo Xuanwo closed this Mar 26, 2021
@Xuanwo Xuanwo deleted the patch-1 branch March 26, 2021 01:41
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.

4 participants