FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat(example): add event usage example with event manager integration by Miou-zora · Pull Request #471 · EngineSquared/EngineSquared · GitHub

feat(example): add event usage example with event manager integration - #471

Merged
Miou-zora merged 3 commits into
mainfrom
add-event-example
Feb 2, 2026
Merged

feat(example): add event usage example with event manager integration#471
Miou-zora merged 3 commits into
mainfrom
add-event-example

Conversation

Miou-zora commented Feb 2, 2026
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Contributor

All is said in the title

Summary by CodeRabbit

  • New Features

    • Added a comprehensive example demonstrating event system usage, showcasing callback registration for immediate and scheduled event handling, callback lifecycle management through registration and unregistration, and event dispatching with configurable timing intervals to illustrate real-world patterns.
  • Chores

    • Added build configuration and compiler settings for the new example project with required dependencies.

Miou-zora requested a review from a team February 2, 2026 20:28
Miou-zora self-assigned this Feb 2, 2026

coderabbitai Bot commented Feb 2, 2026
edited
Loading

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

Adds a new example program demonstrating the event system with an EventExample struct, showing event registration, callback execution, and event publishing. Includes a corresponding XMake build configuration with project dependencies and compilation rules.

Changes

Cohort / File(s) Summary
Event Usage Example
examples/event_usage/src/main.cpp, examples/event_usage/xmake.lua
New example program demonstrating event system usage with immediate and scheduled callbacks, event registration/unregistration, and event publishing. Build configuration defines project dependencies (entt, gtest, glm, glfw, glew, spdlog, fmt, joltphysics, stb) and debug/release build rules.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 An event has hopped into our warren of code,
With callbacks that bind, and flows that have flowed,
I bounce with delight at this example so clear,
Where messages dance from sender to ear,
A timely tale told—hooray, hooray! 🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 1 ❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically summarizes the main change: adding a new example that demonstrates event usage with event manager integration, which matches the changeset perfectly.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch add-event-example

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Miou-zora merged commit 8107359 into main Feb 2, 2026
0 of 5 checks passed
Miou-zora deleted the add-event-example branch February 2, 2026 20:36

sonarqubecloud Bot commented Feb 2, 2026

Copy link
Copy Markdown

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL