Skip to content

chore: A number of minor refactors.#2

Merged
kcexn merged 8 commits intomainfrom
develop
Nov 12, 2025
Merged

chore: A number of minor refactors.#2
kcexn merged 8 commits intomainfrom
develop

Conversation

@kcexn
Copy link
Owner

@kcexn kcexn commented Nov 12, 2025

No description provided.

- Refactor the run() method to be owned by the async_context as
it doesn't depend on anything thread related.
- Moved the isr() method from the context_thread to the
async_context as it has no thread-based dependencies.
- Also updated the documentation for the interrupt
service routine.
- Reordered the async_context elements to be in order of size.
- Moved type information out of context_thread and into
async_context.
- Updated the public API in the umbrella header.
- Update the version number in CMakeLists.txt
- Remove tests and coverage config for the docs workflow.
- Rename the CI caches to fix CI builds.
@kcexn kcexn merged commit a094dee into main Nov 12, 2025
3 checks passed
@codacy-production
Copy link

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
+0.00% (target: -1.00%) 100.00%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (8f6131d) 267 266 99.63%
Head commit (a094dee) 266 (-1) 265 (-1) 99.62% (+0.00%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#2) 22 22 100.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant