Skip to content

Examples #79

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 15 commits into from
Jun 16, 2025
Merged

Examples #79

merged 15 commits into from
Jun 16, 2025

Conversation

andreiavrammsd
Copy link
Owner

@andreiavrammsd andreiavrammsd commented Jun 16, 2025 •
edited
Loading

  • Add semaphore example
  • Add concurrent map filter example
  • Refine readme examples
  • Fix markdown issues
  • Set CMAKE_CXX_EXTENSIONS

@ghost
Copy link

ghost commented Jun 16, 2025 •
edited by ghost
Loading

Pull Request Revisions

RevisionDescription
r6
Refactored output and error handlingUpdated multiple example files with consistent output formatting, explicit expected values, and improved error reporting
r5
Updated example code comments, sleepModified comments in concurrent_map_filter.cpp and semaphore.cpp, adjusted sleep duration, and added EBO comment
r4
Replaced assertions with error handlingModified examples and tests to replace assert with explicit error handling using if statements, std::cerr, and std::terminate()
r3
Refactored concurrent map-filter exampleReplaced merge_channels.cpp with a new concurrent_map_filter.cpp example demonstrating a more complex concurrent data processing pipeline with multiple stages
r2
Updated CMake and example configurationsModified GitHub workflow, CMake example targets, README formatting, and semaphore example code structure
r1
Multiple project configuration and documentation updatesUpdated CMake settings, README formatting, example code, added new semaphore example, and refined clang-tidy configuration
☑️ AI review skipped after 5 revisions, comment with `/review` to review again
Help React with emojis to give feedback on AI-generated reviews:
  • 👍 means the feedback was helpful and actionable
  • 👎 means the feedback was incorrect or unhelpful
💬 Replying to feedback with a comment helps us improve the system. Your input also contributes to shaping future interactions with the AI reviewer.

We'd love to hear from you—reach out anytime at team@review.ai.

ghost
ghost approved these changes Jun 16, 2025
View reviewed changes
@codecov Codecov
Copy link

codecov bot commented Jun 16, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (6799006) to head (786ccf9).
Report is 1 commits behind head on master.

Additional details and impacted files 
@@            Coverage Diff            @@
##            master       #79   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines          110       110           
  Branches         6         6           
=========================================
  Hits           110       110

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

ghost
ghost approved these changes Jun 16, 2025
View reviewed changes
ghost
ghost approved these changes Jun 16, 2025
View reviewed changes
@andreiavrammsd andreiavrammsd merged commit d4667ca into master Jun 16, 2025
11 checks passed
@andreiavrammsd andreiavrammsd deleted the examples branch June 16, 2025 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer
Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@andreiavrammsd