Skip to content
This repository was archived by the owner on Jan 15, 2024. It is now read-only.

Remove outdated comment #943

Merged
merged 1 commit into from
Sep 24, 2019
Merged

Remove outdated comment #943

merged 1 commit into from
Sep 24, 2019

Conversation

leezu
Copy link
Contributor

@leezu leezu commented Sep 24, 2019

Remove outdated comment in test file.

#710 forgot to delete it

cc @dmlc/gluon-nlp-team

@leezu leezu requested a review from a team as a code owner September 24, 2019 19:14
@codecov
Copy link

codecov bot commented Sep 24, 2019

Codecov Report

❗ No coverage uploaded for pull request head (updatecomment@75e4330). Click here to learn what that means.
The diff coverage is n/a.

@mli
Copy link
Member

mli commented Sep 24, 2019

Job PR-943/1 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-943/1/index.html

@szha szha merged commit ad93f1f into dmlc:master Sep 24, 2019
@leezu leezu deleted the updatecomment branch September 25, 2019 09:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
1 more reviewer

@eric-haibin-lin eric-haibin-lin eric-haibin-lin approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@leezu @mli @eric-haibin-lin @szha