Skip to content

Ci/2026 09 03/pin fixtures and repair floors - #35

Merged
janakhpon merged 2 commits into
mainfrom
ci/2026-09-03/pin-fixtures-and-repair-floors
Sep 4, 2026
Merged

Ci/2026 09 03/pin fixtures and repair floors#35
janakhpon merged 2 commits into
mainfrom
ci/2026-09-03/pin-fixtures-and-repair-floors

Conversation

@janakhpon

Copy link
Copy Markdown
Member

What changed

Why

How it was verified

  • pnpm --filter ./apps/web test
  • pnpm --filter ./apps/web run lint
  • cd services/feedback && go vet ./... && go test ./... (if Go changed)
  • Android unit tests, if Android changed: cd apps/android && ./gradlew testDebugUnitTest
    (needs JAVA_HOME on Android Studio's JBR; see apps/android/README.md).
    CI does not run them, so this is the only gate on that suite.
  • iOS MonOcrCore tests, if iOS changed: cd apps/ios && sh Scripts/swift-test.sh.
    The ios-core job runs these on every push, so this is a faster copy of a
    gate that does exist. The app target is still not built anywhere.

Claims

- remove the stray `name: release-cli.yml` key and its orphaned comment block

YAML forbids duplicate keys at one level. A permissive parser keeps the last and
carries on; GitHub can reject the file outright, in which case the workflow never
registers and a `cli/v*` tag would look ignored rather than failed.

Checked the other twelve workflow files in the workspace with a duplicate-key
loader: none affected.
- the Play Store and App Store links both 404; neither app is published
- apps/web pointed at MonDevHub/ocr-android and ocr-ios, which moved here

The Android bullet claimed availability outright, with no caveat, and the link
404s. The iOS one at least said 'pending review' but still handed the reader a
dead URL. Both now name the directory to build from instead.

The two ocr-* repositories were folded into this monorepo; monocr-onnx's README
has recorded them as 404 since 2026-08-15 and apps/web still linked them.

The translated READMEs keep their own wording — only the dead URLs were unwrapped,
because writing new Mon or Burmese status text is not something this change can
verify. A native reader should still check that the surrounding sentences read
correctly now.

The shared translation sheet is unchanged. It returns 401 to an anonymous
fetch by design; access is granted on request.

Every remaining link checked: 20 in tracked markdown, all resolving apart from
that sheet and a reserved example.com placeholder in a code block.
@janakhpon
janakhpon merged commit b50dfab into main Sep 4, 2026
12 of 17 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