Skip to content

Refactor/Adapt ClassRegistry from kajson, remove sandbox manager#103

Merged
lchoquel merged 9 commits intodevfrom
refactor/Sandbox
Jun 19, 2025
Merged

Refactor/Adapt ClassRegistry from kajson, remove sandbox manager#103
lchoquel merged 9 commits intodevfrom
refactor/Sandbox

Conversation

@lchoquel
Copy link
Copy Markdown
Member

@lchoquel lchoquel commented Jun 19, 2025

📝 Description

  • Adapt to new ClassRegistry from kajson, used with dependency injection
  • Remove unused sandbox component, adapt logging, pretty printing, error reporting

🔄 Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 💥 Breaking change
  • 📚 Documentation update
  • 🧹 Code refactor
  • ⚡ Performance improvement
  • ✅ Test update

@lchoquel lchoquel requested a review from thomashebrard June 19, 2025 11:46
@lchoquel lchoquel marked this pull request as ready for review June 19, 2025 11:47
@lchoquel lchoquel changed the title Refactor/sandbox Refactor/Adapt ClassRegistry from kajson, remove sandbox manager Jun 19, 2025
thomashebrard
thomashebrard previously approved these changes Jun 19, 2025
@lchoquel lchoquel merged commit 08bbe33 into dev Jun 19, 2025
14 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 19, 2025
@lchoquel lchoquel deleted the refactor/Sandbox branch June 19, 2025 19:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants