Commit 4cd059d
authored
fix(ci): correct fastlane metadata path; park the unsupported bs listing (#157)
## What
Commit 1: make the Play upload reach the API correctly.
- Point `fastlane supply --metadata_path` at `play/metadata/android`
instead of `play/metadata` in `release.yml`, `play-listing.yml`, and the
`play/README.md` example. Supply treats every child of `metadata_path`
as a Play locale, so the old path made it upload a single bogus locale
named `android` (`Invalid request` from the Play API, first surfaced
once the 1.0.1 run got past the disabled-API error).
- Move `play/metadata/android/bs/` to `play/metadata/unsupported/bs/`.
Google Play does not offer Bosnian (`bs`) as a store-listing language,
so supply's upload for it is rejected outright. The translated listing
content is parked rather than deleted; in-app Bosnian is unaffected.
Commit 2: make sure a release can never ship half-empty again.
- `check_play_metadata.py` hard-fails when `EXPECTED_VERSION_CODE` is
set and any locale lacks `changelogs/<code>.txt`; `release.yml` now
exposes the tag-derived versionCode as a job output and passes it to the
lint, so a tag without release notes stops before the Play upload.
`play-listing.yml` doesn't set it, so listing-only syncs are unaffected.
- The flatten step (both workflows) now also flattens `images/icon/` to
Supply's `images/icon.png`, so the 512x512 store icon uploads
automatically once it's committed; previously only `featureGraphic/` was
flattened and a committed icon would have been silently ignored.
- Renamed the never-shipped 1.0.0 release notes (`changelogs/10000.txt`)
to `10001.txt` in all five locales: no 1.0.0 tag exists, so 1.0.1 is the
initial public release and the "Initial release." copy is its release
copy.
- README: documented the versionCode scheme, the
release-notes-before-tagging rule, and the icon/featureGraphic directory
convention.
## Testing
- Play Listing Sync (`validate_only=true`, images included): fails on
main (`android - Invalid request`, runs 31900131748 / 31900249999),
**Successfully validated the upload to Google Play** on this branch for
all 5 locales (runs 31900400508, 31900845136, the latter with the new
flatten + lint in place). Validate-only builds the full edit server-side
and discards it; no binary uploaded.
- Lint locally: no env = 5 icon warnings, exit 0;
`EXPECTED_VERSION_CODE=10001` = exit 0; `EXPECTED_VERSION_CODE=10002` =
5 missing-changelog errors, exit 1.
- Flatten loop locally with a simulated `images/icon/` PNG: produces
`featureGraphic.png` for all 5 locales plus `icon.png` where present.1 parent 5d0c009 commit 4cd059d
42 files changed
Lines changed: 85 additions & 19 deletions
File tree
- .github/workflows
- play
- metadata
- android
- de-DE/changelogs
- en-US/changelogs
- es-ES/changelogs
- fr-FR/changelogs
- pt-BR/changelogs
- unsupported/bs
- changelogs
- images
- phoneScreenshots
- sevenInchScreenshots
- tenInchScreenshots
- scripts
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
52 | 52 | | |
53 | 53 | | |
54 | 54 | | |
55 | | - | |
| 55 | + | |
56 | 56 | | |
57 | | - | |
58 | | - | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
59 | 60 | | |
60 | 61 | | |
61 | | - | |
| 62 | + | |
62 | 63 | | |
63 | 64 | | |
64 | | - | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
65 | 68 | | |
66 | | - | |
| 69 | + | |
67 | 70 | | |
68 | 71 | | |
69 | 72 | | |
| |||
82 | 85 | | |
83 | 86 | | |
84 | 87 | | |
85 | | - | |
| 88 | + | |
86 | 89 | | |
87 | 90 | | |
88 | 91 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
126 | 126 | | |
127 | 127 | | |
128 | 128 | | |
| 129 | + | |
| 130 | + | |
129 | 131 | | |
130 | 132 | | |
131 | 133 | | |
| |||
321 | 323 | | |
322 | 324 | | |
323 | 325 | | |
| 326 | + | |
| 327 | + | |
324 | 328 | | |
325 | 329 | | |
326 | | - | |
| 330 | + | |
327 | 331 | | |
328 | 332 | | |
329 | | - | |
330 | | - | |
| 333 | + | |
| 334 | + | |
| 335 | + | |
331 | 336 | | |
332 | 337 | | |
333 | | - | |
| 338 | + | |
334 | 339 | | |
335 | 340 | | |
336 | | - | |
| 341 | + | |
| 342 | + | |
| 343 | + | |
337 | 344 | | |
338 | 345 | | |
339 | 346 | | |
| |||
359 | 366 | | |
360 | 367 | | |
361 | 368 | | |
362 | | - | |
| 369 | + | |
363 | 370 | | |
364 | 371 | | |
365 | 372 | | |
| |||
File renamed without changes.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
File renamed without changes.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
File renamed without changes.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
File renamed without changes.
0 commit comments