Skip to content

Combine string containers#930

Merged
maddenp-cu merged 4 commits into
ufs-community:mainfrom
maddenp-cu:combine-string-containers
Jun 12, 2026
Merged

Combine string containers#930
maddenp-cu merged 4 commits into
ufs-community:mainfrom
maddenp-cu:combine-string-containers

Conversation

@maddenp-cu

Copy link
Copy Markdown
Collaborator

Synopsis

To avoid confusion about where to define string constants, combine the EC and STR string containers. For now, the FORMAT container remains, since its class offers some helper methods specific to data formats.

Type

  • Code maintenance (refactoring, etc. without behavior change)

Impact

  • This is a non-breaking change (existing functionality continues to work as expected)

Checklist

  • I have added myself and any co-authors to the PR's Assignees list.
  • I have reviewed the documentation and have made any updates necessitated by this change.
  • Where helpful, I have written comments in this PR's Files changed view to assist reviewers.

@maddenp-cu maddenp-cu self-assigned this Jun 12, 2026
@maddenp-cu maddenp-cu marked this pull request as ready for review June 12, 2026 17:39
@maddenp-cu maddenp-cu merged commit 41f00eb into ufs-community:main Jun 12, 2026
2 checks passed
@maddenp-cu maddenp-cu deleted the combine-string-containers branch June 12, 2026 18:41
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