Skip to content

refactor: migrate electrochem apps to owned packages#12

Merged
Pluze merged 1 commit into
mainfrom
codex/app-namespace-migration
Jun 5, 2026
Merged

refactor: migrate electrochem apps to owned packages#12
Pluze merged 1 commit into
mainfrom
codex/app-namespace-migration

Conversation

@Pluze

@Pluze Pluze commented Jun 5, 2026

Copy link
Copy Markdown
Owner

Scope

  • Move the five electrochem apps into app-owned folders and packages while preserving public launch command names.
  • Remove the old electrochemWorkflow/private workflow and runner entry points from the app surface.
  • Update tests, project guardrails, docs, and MATLAB Project path metadata for the migrated layout.

Validation

  • scripts/run_matlab_tests.sh testProject
  • scripts/run_matlab_tests.sh checkProject
  • scripts/run_matlab_tests.sh testAppsElectrochem
  • scripts/run_matlab_tests.sh testAppsElectrochemGui
  • scripts/run_matlab_tests.sh testAppsSmokeGui
  • scripts/run_matlab_tests.sh test
  • scripts/run_matlab_tests.sh checkStyle

Unverified

  • Manual MATLAB GUI workflow feel and real file-selection/export interactions were not exercised; automated GUI checks are structural launch/layout/callback coverage only.

Follow-up

  • Continue with the DIC migration, then wearable, before tightening the global private/workflow ban.

@Pluze Pluze merged commit e9e087a into main Jun 5, 2026
7 checks passed
@Pluze Pluze deleted the codex/app-namespace-migration branch June 5, 2026 19:52
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