Unit tests #32
Unit tests #32
Conversation
|
Awesome. So nice too that you added a
Moving forward, having automated tests to vet PRs should be a real boon. Cheers! |
| @@ -1,4 +1,4 @@ | |||
| language: generic | |||
| language: c | |||
pq
Apr 7, 2016
Contributor
Yay!
Do we want to specify compiler? (Or have a matrix?)
For example:
compiler:
- clang
- gcc
Yay!
Do we want to specify compiler? (Or have a matrix?)
For example:
compiler:
- clang
- gcc
samdoshi
Apr 7, 2016
Author
Collaborator
I made it generic initially as we bring our own toolchain!
Multiple compilers seems a bit overkill to me, for now the only target that matters is the hardware (really we should install gcc-4.4, but baby steps...).
I made it generic initially as we bring our own toolchain!
Multiple compilers seems a bit overkill to me, for now the only target that matters is the hardware (really we should install gcc-4.4, but baby steps...).
pq
Apr 7, 2016
Contributor
Sounds good. If gcc is the expectation may not hurt to specify it but no biggie either way!
Sounds good. If gcc is the expectation may not hurt to specify it but no biggie either way!
|
|
||
| MaxEmptyLinesToKeep: 2 | ||
| --- | ||
|
|
pq
Apr 7, 2016
Contributor
Is there a reason not to put these at the root dir? IOW do we not want to format the other bits too?
Is there a reason not to put these at the root dir? IOW do we not want to format the other bits too?
samdoshi
Apr 7, 2016
Author
Collaborator
Basically, yes, there is the libavr32 submodule which is used by other modules.
I'm just being cautious, if everyone gets on with using an editorconfig and clang-format then we'll do libavr32 as well, and then move these files to the root.
Basically, yes, there is the libavr32 submodule which is used by other modules.
I'm just being cautious, if everyone gets on with using an editorconfig and clang-format then we'll do libavr32 as well, and then move these files to the root.
pq
Apr 7, 2016
Contributor
Oh, right. Cool. Thanks for the clarification!
Oh, right. Cool. Thanks for the clarification!
| [*.{h,hpp,c,cpp}] | ||
| indent_style = space | ||
| indent_size = 4 | ||
| trim_trailing_whitespace = true |
pq
Apr 7, 2016
Contributor
Same question here.
Same question here.
Adds unit tests using the greatest library.
Moves the simulator up a directory
Also commit 7c66926 changes the prototype for
parseso as to remove the globaltempvariable.