Import Fatras
This imports the current master from acts-fatras into the Fatras
directory. To keep this separate from the core plugins the include files are located in ActsFatras/
and all code is in the ActsFatras
namespace. Tests are available in the Tests/UnitTests/Fatras
directory.
The outstanding issues for Fatras will be moved to the core repository once this has been merge (contrary to previous plans of fixing all issues first).
Contains some small modifications to simplify adding unit/integration tests with additional libraries.
Preparation for #686.
Merge request reports
Activity
changed milestone to %0.16.00
added Component - Fatras Improvement Major labels
mentioned in issue #686
added 22 commits
Toggle commit list- Resolved by Paul Gessinger
- Resolved by Paul Gessinger
- Resolved by Paul Gessinger
Ah, can you update the global readme as well?
Another question: FATRAS unit tests coverage is not included. I think this is because we disable it in the coverage exclusion patterns. This is also the case with the plugins. Maybe we need to devise a way how cmake can communicate which plugins are enabled, and those plugins and their tests are then taken into account in the coverage calculation.
Hm, maybe it isn't.
Edited by Paul Gessingeradded 2 commits
The CI should now build Fatras as well. Some of the tests fail and I will have to look into that. I do not think I will have time for that today.
Edited by Moritz Kiehnadded 13 commits
-
4cf8357f...404a6024 - 6 commits from branch
master
- ce27a27b - cmake: allow extending common libraries for tests
- b0ad760b - cmake: extend component log lines
- 4e4aaae5 - integrate current Fatras master
- b568cccd - cmake: add fatras
- 85260464 - cmake: reorder options for clarity
- de0e5bbd - update README and getting started
- 8c2c5468 - ci: build fatras
Toggle commit list-
4cf8357f...404a6024 - 6 commits from branch
assigned to @pagessin
enabled an automatic merge when the pipeline for 8c2c5468 succeeds
mentioned in commit f8a3c0f7
mentioned in merge request acts-framework!203 (merged)