Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove outdated todo comment #10850

Merged
merged 1 commit into from
Jun 30, 2020
Merged

Conversation

ryoqun
Copy link
Member

@ryoqun ryoqun commented Jun 30, 2020

Problem

Some comment from #9527 is outdated.

Summary of Changes

Remove it because #10348 has already been landed.

@ryoqun ryoqun added automerge Merge this Pull Request automatically once CI passes v1.2 labels Jun 30, 2020
@codecov
Copy link

codecov bot commented Jun 30, 2020

Codecov Report

Merging #10850 into master will not change coverage.
The diff coverage is 100.0%.

@@           Coverage Diff           @@
##           master   #10850   +/-   ##
=======================================
  Coverage    81.9%    81.9%           
=======================================
  Files         303      303           
  Lines       71079    71079           
=======================================
  Hits        58224    58224           
  Misses      12855    12855           

@mergify mergify bot merged commit cabedc0 into solana-labs:master Jun 30, 2020
mergify bot pushed a commit that referenced this pull request Jun 30, 2020
(cherry picked from commit cabedc0)
mergify bot added a commit that referenced this pull request Jun 30, 2020
(cherry picked from commit cabedc0)

Co-authored-by: Ryo Onodera <ryoqun@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge this Pull Request automatically once CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant