-
Notifications
You must be signed in to change notification settings - Fork 4
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
Openandorder refactor #36
Conversation
06ecb8e
to
9f2f033
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any way to test the introduced features, like argument parsing?
At the moment I'd say there are more important things to do than unit testing the argument parsing that has some system tests, but could be persuaded otherwise! |
6186828
to
e86d7b9
Compare
Rename Add argument parser class Extract function Delete non required standard specification Include stdecept Remove unused header file Refactor saveoutput Refactor openandorder Extract class [skip actions] Extract matrix collections Include stdexcept More stdexcept More function extraction include cstring Extract functions Extract class Include stdexcept Self review changes tmp t
e86d7b9
to
3bd56d0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't looked at detail into ftdd_grid_initialiser.cpp
, matrix_collection.cpp
, or openandorder.cpp
as they seem copy/pasted and changing the code is outside scope of this PR. Otherwise looks 👍 with some minor suggestions
As a general comment, would be good to get a linter to enforce consisted code style asap. I'm happy to open an issue for that?
Oh, also a rebase onto main to get a CI run in would be good |
Co-authored-by: David Stansby <dstansby@gmail.com>
Refactors
openandorder
. This PR adds: