Skip to content

Fixing errors - #20

Open
Matographo wants to merge 3 commits into
mainfrom
fix/fixing-potential-errors
Open

Fixing errors#20
Matographo wants to merge 3 commits into
mainfrom
fix/fixing-potential-errors

Conversation

@Matographo

Copy link
Copy Markdown
Contributor

Summary

Looking for fixable potential Bugs

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation (wiki or in-repo)
  • Build / CI / tooling
  • Refactor (no behavior change)

How was this tested?

  • make test
  • make all (full local QA pipeline)
  • Other (describe below)

User-visible changes

None

Breaking changes

None

Documentation

  • Wiki updated (required for user-visible CLI, DSL, config, cache, or UI changes)
  • CHANGELOG.md updated (if appropriate)
  • In-repo docs updated (README.md, docs/, as needed)
  • Not needed (internal-only change)

Checklist

  • New source and test files are registered in CMake where needed
  • DSL or parser changes include fuzz corpus updates when appropriate (tests/fuzz/corpus/lua_dsl/)
  • Line coverage on src/ ≥ 85% (make coverage)
  • No unrelated drive-by refactors mixed into this change
  • I have read CONTRIBUTING.md

sol::nil is not available in the pinned sol2 version; align with the existing nil checks used elsewhere in the Lua plugin layer.
Sign up for freeto 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

@Matographo