Skip to content

[draft] Refactor Value#100

Draft
CUB3D wants to merge 16 commits into
masterfrom
value_refactor
Draft

[draft] Refactor Value#100
CUB3D wants to merge 16 commits into
masterfrom
value_refactor

Conversation

@CUB3D

@CUB3D CUB3D commented May 25, 2026

Copy link
Copy Markdown
Collaborator

(This is a draft until #99 is merged)

Rework the Value enum to:

  • Give names to the associated data and allow documenting them
  • Allow adding / removing fields to said data, without requiring as much refactoring

The id field is intentionally kept outside the ObjectValue etc struct to make it harder to miss if added/removed from a structure.

@CUB3D CUB3D force-pushed the value_refactor branch from db48622 to 212f6a6 Compare May 28, 2026 21:50
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.

1 participant