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

Use conventional special self notation #11990

Merged
merged 1 commit into from
Sep 2, 2020

Conversation

ryoqun
Copy link
Contributor

@ryoqun ryoqun commented Sep 2, 2020

Well, when I moved this from MessageProcessor at #11750, this bad practice began. And quickly got widespread. ;)

@ryoqun ryoqun added the automerge Merge this Pull Request automatically once CI passes label Sep 2, 2020
@codecov
Copy link

codecov bot commented Sep 2, 2020

Codecov Report

Merging #11990 into master will decrease coverage by 0.0%.
The diff coverage is 100.0%.

@@            Coverage Diff            @@
##           master   #11990     +/-   ##
=========================================
- Coverage    82.2%    82.2%   -0.1%     
=========================================
  Files         333      333             
  Lines       78222    78222             
=========================================
- Hits        64328    64309     -19     
- Misses      13894    13913     +19     

@mergify mergify bot merged commit 46aac48 into solana-labs:master Sep 2, 2020
@ryoqun ryoqun added the v1.3 label Sep 2, 2020
mergify bot pushed a commit that referenced this pull request Sep 2, 2020
mergify bot added a commit that referenced this pull request Sep 2, 2020
(cherry picked from commit 46aac48)

Co-authored-by: Ryo Onodera <[email protected]>
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