Skip to content

Conversation

@maskit
Copy link
Member

@maskit maskit commented Jun 12, 2017

No description provided.

@maskit maskit added the Cleanup label Jun 12, 2017
@maskit maskit added this to the 8.0.0 milestone Jun 12, 2017
@maskit maskit self-assigned this Jun 12, 2017
@masaori335
Copy link
Contributor

I'm just curious that clang-tidy's modernize-use-override is not applied to all files?

@jpeach
Copy link
Contributor

jpeach commented Jun 12, 2017 via email

@masaori335
Copy link
Contributor

masaori335 commented Jun 12, 2017

OK, got it. Thanks.

Copy link
Contributor

@zwoop zwoop left a comment

Choose a reason for hiding this comment

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

+1

@maskit maskit merged commit 3580ec8 into apache:master Jun 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants