Add docs and checks#16
Conversation
xalbertoisorna
commented
May 21, 2026
xalbertoisorna
commented
- Update .gitignore to include _build and dist directories.
- Enhance Jenkinsfile with VX4 build stage.
- Create exclude_patterns.inc for documentation generation.
- Add lib_unity.rst for project documentation.
- Introduce CMakeLists.txt and main.c for basic example tests.
- Implement test_basic.c with unit tests for addition, division, and sine functions.
- Modify CMakeLists.txt for uut_and_tests to target XK-EVK-XU316.
- Add settings.yml for project configuration and documentation settings.
xalbertoisorna
force-pushed
the
feature/docs-and-checks
branch
from
5681c14 to
5051b09
Compare
Allan-xmos
left a comment
Allan-xmos
left a comment
There was a problem hiding this comment.
One small version to fix, but otherwise good
| string( | ||
| name: 'XMOSDOC_VERSION', | ||
| defaultValue: 'v8.2.0', | ||
| defaultValue: 'v8.1.0', |
There was a problem hiding this comment.
This was a mistake from my previous PR (that one did not have docs), latest version is 8.1.0
| :description: A test framework for C, adapted from the Unity test framework | ||
| :category: General Purpose | ||
| :keywords: | ||
| :devices: xcore-200, xcore.ai |
There was a problem hiding this comment.
I assume that the omission of xcore-400 comes from a recognition that the processor is not yet generally available. On the other hand, if it's an oversight, I wanted to flag it.
There was a problem hiding this comment.
I note that this example is not currently built for XU416. As long as that's the intention, and a later PR will add xcore-400 support, that's not a problem, just an observation.
There was a problem hiding this comment.
Adding vx4 build now, that would make sense in this PR.
There was a problem hiding this comment.
I note that this example is not currently built for XU416. As long as that's the intention, and a later PR will add xcore-400 support, that's not a problem, just an observation.
There was a problem hiding this comment.
Adding vx4 build now, that would make sense in this PR.
3 participants
