Skip to content

feat(scripting): add depends function in lua#1575

Open
mcanais wants to merge 1 commit intomainfrom
1519-expose-cubosdepends-in-lua
Open

feat(scripting): add depends function in lua#1575
mcanais wants to merge 1 commit intomainfrom
1519-expose-cubosdepends-in-lua

Conversation

@mcanais
Copy link
Contributor

@mcanais mcanais commented Sep 26, 2025

Description

Added cubos.depends() function to the lua api which allows to dynamically import the contents of a plugin (components, resources and relations) and return them as a table in lua.

Each of the contents is represented as a userdata (pointing to const reflection::Type*) with a metatable corresponding to its kind (either component, resource or relation).

Checklist

  • Self-review changes.
  • Evaluate impact on the documentation.
  • Ensure test coverage.
  • Write new samples.
  • Add entry to the changelog's unreleased section.

@mcanais mcanais requested a review from RiscadoA as a code owner September 26, 2025 12:09
@mcanais mcanais linked an issue Sep 26, 2025 that may be closed by this pull request
@mcanais mcanais requested review from mkuritsu and removed request for RiscadoA September 26, 2025 12:09
@github-actions
Copy link
Contributor

github-actions bot commented Sep 26, 2025

PR Preview Action v1.6.2

🚀 View preview at
https://GameDevTecnico.github.io/cubos/preview/pr-1575/

Built to branch gh-pages at 2025-09-26 14:41 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@codecov
Copy link

codecov bot commented Sep 26, 2025

Codecov Report

❌ Patch coverage is 56.25000% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.76%. Comparing base (3fb5778) to head (0c9754f).
⚠️ Report is 12 commits behind head on main.

Files with missing lines Patch % Lines
core/src/ecs/cubos.cpp 56.25% 7 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1575   +/-   ##
=======================================
  Coverage   51.76%   51.76%           
=======================================
  Files         501      501           
  Lines       27267    27281   +14     
  Branches     2550     2553    +3     
=======================================
+ Hits        14114    14123    +9     
- Misses      13153    13158    +5     

☔ 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.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ Clang-Tidy found issue(s) with the introduced code (1/1)

@mcanais mcanais force-pushed the 1519-expose-cubosdepends-in-lua branch from 6479859 to 0c9754f Compare September 26, 2025 14:39
@mcanais mcanais self-assigned this Sep 26, 2025
@mcanais mcanais enabled auto-merge (rebase) September 26, 2025 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Expose cubos.depends in Lua

1 participant