Skip to content

refactor: rename apps to demos - #19

Merged
tela merged 1 commit into
mainfrom
chore/rename-apps-to-demos
Nov 7, 2026
Merged

tela merged 1 commit into
mainfrom
chore/rename-apps-to-demos

Conversation

@tela

@tela tela commented Nov 7, 2026

Copy link
Copy Markdown
Contributor

Motivation

samples/ holds third-party apps run against the app emulators, and apps/ held our own staged demos. The two trees read as one family. The repo now has two children, demos/ and samples/.

Changes

  • Rename apps/ to demos/ and web/apps/ to web/demos/, so the Go and UI trees stay parallel.
  • Update the Go module path to .../demos/acme-commerce-lenders, plus go.work, the web workspace globs (web/pnpm-workspace.yaml, web/biome.json) and the lockfile importer key.
  • Update Makefile, .gitignore, .no-mistakes.yaml and .github/workflows/ci.yml to the new path.
  • Update README.md, web/README.md and samples/README.md. samples/ keeps its meaning.

No behavior changes. The comments in web/pnpm-workspace.yaml that mention apps/ refer to other repos and are untouched.

Tests

make vet                                 ok
make test                                ok (all packages, including demos/acme-commerce-lenders)
make check-kits                          ok
make check-samples                       samples in ../samples are well formed
make build                               builds bin/acme-commerce-lenders-kit
pnpm --dir web install --frozen-lockfile ok
pnpm --dir web lint                      Checked 44 files, no fixes applied
pnpm --dir web type-check                Done
pnpm --dir web test                      9 files, 45 tests passed
make web APP=acme-commerce-lenders       builds and copies the UI

Related

Follows localstack-samples/localstack-app-emulators#17 and #18.

samples/ holds third-party apps and apps/ held our demos, so the two
trees read as one family. Rename apps/ and web/apps/ to demos/ and
update the Go module path, workspace files, CI and docs to match.
@tela
tela merged commit 261d0c4 into main Nov 7, 2026
2 checks passed
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