Fix minor typos
ClosedPublic

Authored by yurchor on Sep 11 2019, 5:22 PM.

Details

Summary

Just a typofix patch.

Test Plan

Needs review because of changing sample files.

Diff Detail

Repository
R55 Cantor
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 16389
Build 16407: arc lint + arc unit
yurchor created this revision.Sep 11 2019, 5:22 PM
Restricted Application added a project: KDE Edu. · View Herald TranscriptSep 11 2019, 5:22 PM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
yurchor requested review of this revision.Sep 11 2019, 5:22 PM
sirgienko requested changes to this revision.Sep 11 2019, 6:23 PM

I think, there isn't need to edit *.ipynb, because they are just an example input data, and also used in tests. So, I think, better revert the .ipynb changes, or update tests content testworksheet.cpp too.

This revision now requires changes to proceed.Sep 11 2019, 6:23 PM

I think, there isn't need to edit *.ipynb, because they are just an example input data, and also used in tests. So, I think, better revert the .ipynb changes, or update tests content testworksheet.cpp too.

But I fixed src/test/worksheet_test.cpp...

Ok. I will revert the changes.

yurchor updated this revision to Diff 65871.Sep 11 2019, 6:33 PM

Remove changes in the tests

sirgienko accepted this revision.Sep 11 2019, 7:47 PM
This revision is now accepted and ready to land.Sep 11 2019, 7:47 PM
This revision was automatically updated to reflect the committed changes.