Skip to content
This repository was archived by the owner on Jun 30, 2022. It is now read-only.

Conversation

@xieofxie
Copy link
Contributor

@xieofxie xieofxie commented Mar 16, 2020

Purpose

What is the context of this pull request? Why is it being done?

Close #3180

Changes

Are there any changes that need to be called out as significant or particularly difficult to grasp? (Include illustrative screenshots for context if applicable.)

Will mirror to template if good

Tests

Is this covered by existing tests or new ones? If no, why not?

Feature Plan

Are there any remaining steps or dependencies before this issue can be fully resolved? If so, describe and link to any relevant pull requests or issues.

Checklist

General

  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated the appropriate tests
  • I have updated related documentation

@xieofxie
Copy link
Contributor Author

One more: should SDK be updated?

@lzc850612
Copy link
Contributor

Let's hold off on updating to rc5 as the official release will come out on Monday.

Copy link
Contributor

@lzc850612 lzc850612 left a comment

Choose a reason for hiding this comment

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

Let's hold off on upadting to rc5 as the official release is coming out on Monday

@xieofxie xieofxie closed this Mar 17, 2020
@xieofxie xieofxie deleted the hualxie/fix_eoc branch March 17, 2020 02:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants