Skip to content

Update execution version#82

Merged
dietmarkuehl merged 4 commits intomainfrom
update-execution-version
Apr 22, 2026
Merged

Update execution version#82
dietmarkuehl merged 4 commits intomainfrom
update-execution-version

Conversation

@dietmarkuehl
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings April 21, 2026 22:11
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the repository to a newer revision of the bemanproject/execution dependency and aligns the task library/tests with API changes by relying on beman::execution::inline_scheduler instead of maintaining a task-local inline scheduler implementation.

Changes:

  • Bump bemanproject/execution FetchContent GIT_TAG.
  • Remove beman::task::detail::inline_scheduler (header + test) and switch call sites/tests to beman::execution::inline_scheduler.
  • Adjust build/test/example CMake lists to drop removed sources/targets.

Reviewed changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated no comments.

Show a summary per file
File Description
CMakeLists.txt Updates the execution dependency GIT_TAG.
include/beman/task/task.hpp Re-exports inline_scheduler from beman::execution and updates includes accordingly.
include/beman/task/detail/task.hpp Drops now-unneeded include of the removed inline scheduler header.
include/beman/task/detail/promise_type.hpp Drops now-unneeded include of the removed inline scheduler header.
include/beman/task/detail/promise_env.hpp Reorders query(...) overloads (no functional change intended).
include/beman/task/detail/inline_scheduler.hpp Removes the task-local inline scheduler implementation.
src/beman/task/CMakeLists.txt Removes the deleted header from the exported header file set.
tests/beman/task/CMakeLists.txt Removes the deleted inline scheduler test from the test list.
tests/beman/task/inline_scheduler.test.cpp Deletes tests targeting the removed task-local inline scheduler.
tests/beman/task/task_scheduler.test.cpp Removes include of the deleted task-local inline scheduler header.
tests/beman/task/state_base.test.cpp Switches scheduler usage to beman::execution::inline_scheduler.
tests/beman/task/scheduler_of.test.cpp Updates expected scheduler type to beman::execution::inline_scheduler.
tests/beman/task/promise_type.test.cpp Switches scheduler usage/initialization to beman::execution::inline_scheduler.
tests/beman/task/promise_base.test.cpp Switches scheduler usage to beman::execution::inline_scheduler.
examples/bulk.cpp Removes a local par tag definition; keeps example compiling standalone but it’s no longer built by default.
examples/CMakeLists.txt Drops bulk from the examples list.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@coveralls
Copy link
Copy Markdown

coveralls commented Apr 21, 2026

Coverage Status

Coverage is 94.059%update-execution-version into main. No base build found for main.

@dietmarkuehl dietmarkuehl merged commit abbb977 into main Apr 22, 2026
36 checks passed
@dietmarkuehl dietmarkuehl deleted the update-execution-version branch April 22, 2026 05:41
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.

3 participants