Skip to content

Clean up DynamicTable utility argument blocks and naming#814

Draft
ehennestad wants to merge 4 commits intobug-fix-dynamic-table-getrowfrom
dynamic-table-utility-argument-blocks
Draft

Clean up DynamicTable utility argument blocks and naming#814
ehennestad wants to merge 4 commits intobug-fix-dynamic-table-getrowfrom
dynamic-table-utility-argument-blocks

Conversation

@ehennestad
Copy link
Copy Markdown
Collaborator

@ehennestad ehennestad commented Apr 14, 2026

Motivation

Clean up the types.util.dynamictable utilities namespace so that the namespace is easier to maintain and so that coding agents produce more readable code.

Changes

  • Use arguments blocks instead of input parsing
  • camelCase variable naming
  • more descriptive variable names
  • consistent indentation
  • clean up whitespace

How to test the behavior?

runtests('+tests/+unit/dynamicTableTest.m');
runtests('+tests/+system/DynamicTableTest.m');

Todo

  •  Agree on scope and necessity of this refactor

Checklist

  • Have you ensured the PR description clearly describes the problem and solutions?
  • Have you checked to ensure that there aren't other open or previously closed Pull Requests for the same change?
  • If this PR fixes an issue, is the first line of the PR description fix #XX where XX is the issue number?

🤖 Generated with Codex

@ehennestad ehennestad changed the title [codex] Clean up DynamicTable utility argument blocks and naming Clean up DynamicTable utility argument blocks and naming Apr 14, 2026
@ehennestad ehennestad changed the base branch from main to bug-fix-dynamic-table-getrow April 14, 2026 21:02
@ehennestad ehennestad changed the base branch from bug-fix-dynamic-table-getrow to main April 14, 2026 21:02
@ehennestad ehennestad changed the base branch from main to bug-fix-dynamic-table-getrow April 14, 2026 21:03
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