Skip to content

Rename embedlayernorm_op_test.cc to embed_layer_norm_op_test.cc#8160

Merged
nkreeger merged 2 commits intomasterfrom
kreeger/eln-cleanup
Jun 26, 2021
Merged

Rename embedlayernorm_op_test.cc to embed_layer_norm_op_test.cc#8160
nkreeger merged 2 commits intomasterfrom
kreeger/eln-cleanup

Conversation

@nkreeger
Copy link
Copy Markdown
Contributor

Matches existing style of other unit test files.

Cleanup from #8124

@nkreeger nkreeger requested a review from a team as a code owner June 25, 2021 23:17
@nkreeger nkreeger requested a review from mrry June 25, 2021 23:18
mrry
mrry previously approved these changes Jun 25, 2021
@snnn
Copy link
Copy Markdown
Contributor

snnn commented Jun 26, 2021

You build failed because you changed two submodules. They are not needed for this PR.

@nkreeger
Copy link
Copy Markdown
Contributor Author

You build failed because you changed two submodules. They are not needed for this PR.

Thanks something accidently snuck in there!

@nkreeger nkreeger merged commit 588511d into master Jun 26, 2021
@nkreeger nkreeger deleted the kreeger/eln-cleanup branch June 26, 2021 02:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants