[android][ios] Upload metadata and screenshots to AppStore / GooglePlay #1035

Merged
root merged 1 commit from rt-upload-metadata into master 2021-08-14 06:54:00 +00:00
Owner

Signed-off-by: Roman Tsisyk roman@tsisyk.com

Signed-off-by: Roman Tsisyk <roman@tsisyk.com>
vng (Migrated from github.com) reviewed 2021-08-07 19:13:39 +00:00
biodranik (Migrated from github.com) requested changes 2021-08-07 21:22:44 +00:00
biodranik (Migrated from github.com) commented 2021-08-07 21:07:16 +00:00

Тут будет проверка на длину?

Тут будет проверка на длину?
biodranik (Migrated from github.com) commented 2021-08-07 21:08:12 +00:00

А валидация на длину будет проходить до мержа в мастер?

А валидация на длину будет проходить до мержа в мастер?
biodranik (Migrated from github.com) commented 2021-08-07 21:10:00 +00:00

Обязательно в src? Почему нельзя, например, android/metadata?

Обязательно в src? Почему нельзя, например, android/metadata?
biodranik (Migrated from github.com) commented 2021-08-07 21:10:46 +00:00

А что тут?

А что тут?
biodranik (Migrated from github.com) commented 2021-08-07 21:14:16 +00:00

Это четвёртое место, где я прошу исправить. Хватит плодить копипасту плохого текста. Все комменты из другого PR валидны и здесь. Можно, кстати, просто было сделать симлинк.

Это четвёртое место, где я прошу исправить. Хватит плодить копипасту плохого текста. Все комменты из другого PR валидны и здесь. Можно, кстати, просто было сделать симлинк.
biodranik (Migrated from github.com) commented 2021-08-07 21:16:06 +00:00

Что за альфа и продакшн?

Что за альфа и продакшн?
biodranik (Migrated from github.com) commented 2021-08-07 21:17:16 +00:00

Я 100% исправлял это в эпл сторе. И ниже текст не тот, что был.

Я 100% исправлял это в эпл сторе. И ниже текст не тот, что был.
biodranik (Migrated from github.com) commented 2021-08-07 21:21:47 +00:00

Разве bash умеет +e внутри функций? Не надо ли назад -e в конце?

Разве bash умеет +e внутри функций? Не надо ли назад -e в конце?
biodranik (Migrated from github.com) commented 2021-08-07 21:22:23 +00:00

Перенос строки будет игнорироваться?

Перенос строки будет игнорироваться?
rtsisyk reviewed 2021-08-08 05:23:24 +00:00
Author
Owner

Скриншоты и какие-то иконки. Вынесено в отдельную репу, чтобы не засирать общую.

Скриншоты и какие-то иконки. Вынесено в отдельную репу, чтобы не засирать общую.
rtsisyk reviewed 2021-08-08 05:24:31 +00:00
Author
Owner

Я это выгрузил из гугл стора 10 часов назад. Задачи править сами тексты в данном PR не стояло.
Симлинк - это в целом примерно то, что я хотел сделать в итоге, чтобы не копипастить.

Я это выгрузил из гугл стора 10 часов назад. Задачи править сами тексты в данном PR не стояло. Симлинк - это в целом примерно то, что я хотел сделать в итоге, чтобы не копипастить.
rtsisyk reviewed 2021-08-08 05:36:38 +00:00
Author
Owner

К сожалению только на длину и будет. Остальные проверки по неведомой причине требуют ключ от аппстора.

К сожалению только на длину и будет. Остальные проверки по неведомой причине требуют ключ от аппстора.
rtsisyk reviewed 2021-08-08 05:37:03 +00:00
Author
Owner

Чек на длину стоит на каждый pull request.

Чек на длину стоит на каждый pull request.
rtsisyk reviewed 2021-08-08 05:38:20 +00:00
Author
Owner

Можем сделать симлинк. Плагин как-то не хотелось бы патчить, если честно.

Мы хотим шарить мету с f-droid?

Можем сделать симлинк. Плагин как-то не хотелось бы патчить, если честно. Мы хотим шарить мету с f-droid?
rtsisyk reviewed 2021-08-08 05:38:52 +00:00
Author
Owner

Поменял в фдроид ПР. Давайте его вольем сначала, потом я эти тексты симлинкну или скопирую (как хотим?)

Поменял в фдроид ПР. Давайте его вольем сначала, потом я эти тексты симлинкну или скопирую (как хотим?)
rtsisyk reviewed 2021-08-08 05:39:51 +00:00
Author
Owner

production - это публичное в сторе
beta - open testing
alpha - closed testing
internal - internal testing

Я пока еще разбираюсь в какой момент changelog заливается. Вроде бы с заливкой апкшки.

production - это публичное в сторе beta - open testing alpha - closed testing internal - internal testing Я пока еще разбираюсь в какой момент changelog заливается. Вроде бы с заливкой апкшки.
rtsisyk reviewed 2021-08-08 05:40:38 +00:00
Author
Owner

Ок, поменяю здесь. Может перезатерло в процессе тестирования скриптов.

Ок, поменяю здесь. Может перезатерло в процессе тестирования скриптов.
rtsisyk reviewed 2021-08-08 05:43:18 +00:00
Author
Owner

Глянь код выше - пришлось python вызвать из оригинального патча. Если есть идеи на как на баше длину строки в юникоде проверить и проигнорировать "\n" в конце - расскажи, я быстрого решения не нашел. Ну разве что вызывать awk/perl, но тут уже и python можно тогда.

len=$(python3 -c "import sys; print(len(sys.stdin.read().rstrip('\n')))" < $1)

Глянь код выше - пришлось python вызвать из оригинального патча. Если есть идеи на как на баше длину строки в юникоде проверить и проигнорировать "\n" в конце - расскажи, я быстрого решения не нашел. Ну разве что вызывать awk/perl, но тут уже и python можно тогда. > len=$(python3 -c "import sys; print(len(sys.stdin.read().rstrip('\n')))" < $1)
rtsisyk reviewed 2021-08-08 05:44:02 +00:00
Author
Owner

Можем поставить -e в конце. Хотелось, чтобы проверялось всё и падало в конце, а не на первой ошибке.

Можем поставить -e в конце. Хотелось, чтобы проверялось всё и падало в конце, а не на первой ошибке.
Author
Owner

Давайте #1035 зальем, я в этом PR тогда Android попробую симликнуть или около того.

Давайте #1035 зальем, я в этом PR тогда Android попробую симликнуть или около того.
biodranik (Migrated from github.com) reviewed 2021-08-08 06:27:06 +00:00
biodranik (Migrated from github.com) commented 2021-08-08 06:27:05 +00:00

Я пишу комменты потому, что не хочу потом снова возвращаться к этой проблеме. На сайте эти штуки исправлял тоже, просто каждый раз видно, как можно сделать ещё лучше.

Я пишу комменты потому, что не хочу потом снова возвращаться к этой проблеме. На сайте эти штуки исправлял тоже, просто каждый раз видно, как можно сделать ещё лучше.
biodranik (Migrated from github.com) reviewed 2021-08-08 06:28:11 +00:00
biodranik (Migrated from github.com) commented 2021-08-08 06:28:11 +00:00

Надо будет разные файлы для разных билдов? А что будет триггерить билды тогда? Может давай сделаем изменение этих файлов чтобы заливало нужный билд?

Надо будет разные файлы для разных билдов? А что будет триггерить билды тогда? Может давай сделаем изменение этих файлов чтобы заливало нужный билд?
biodranik (Migrated from github.com) reviewed 2021-08-08 06:56:31 +00:00
biodranik (Migrated from github.com) commented 2021-08-08 06:56:31 +00:00

Шарить не хотим, но решение прятать метаданные глубоко в исходники выглядит очень странным как минимум.

Шарить не хотим, но решение прятать метаданные глубоко в исходники выглядит очень странным как минимум.
biodranik (Migrated from github.com) reviewed 2021-08-08 06:57:02 +00:00
biodranik (Migrated from github.com) commented 2021-08-08 06:57:01 +00:00

Симлинк сможет помочь, если плагин штатно не умеет.

Симлинк сможет помочь, если плагин штатно не умеет.
biodranik (Migrated from github.com) reviewed 2021-08-08 07:22:02 +00:00
biodranik (Migrated from github.com) commented 2021-08-08 07:22:02 +00:00

${#var} или wc -m показывают длину. Для игнора \n достаточно читать через read. При наличии верной UTF-8 локали, конечно же.

`${#var}` или `wc -m` показывают длину. Для игнора \n достаточно читать через read. При наличии верной UTF-8 локали, конечно же.
biodranik (Migrated from github.com) reviewed 2021-08-08 07:23:54 +00:00
biodranik (Migrated from github.com) commented 2021-08-08 07:23:54 +00:00

Переносы строки нужно игнорить только в однострочных метаданных, конечно же.

Переносы строки нужно игнорить только в однострочных метаданных, конечно же.
biodranik (Migrated from github.com) reviewed 2021-08-08 10:20:00 +00:00
biodranik (Migrated from github.com) commented 2021-08-08 10:20:00 +00:00

Я к тому, что этот +e не выключит ли ошибки ПОСЛЕ вызова функции?

Я к тому, что этот +e не выключит ли ошибки ПОСЛЕ вызова функции?
rtsisyk reviewed 2021-08-08 10:57:26 +00:00
Author
Owner

AppStore, GooglePlay и F-Droid теперь проверяются в Pull Requests при изменении соответствующих директорий:

https://github.com/organicmaps/organicmaps/runs/3273269781?check_suite_focus=true#step:3:1
https://github.com/organicmaps/organicmaps/runs/3273269780?check_suite_focus=true#step:3:1

AppStore, GooglePlay и F-Droid теперь проверяются в Pull Requests при изменении соответствующих директорий: https://github.com/organicmaps/organicmaps/runs/3273269781?check_suite_focus=true#step:3:1 https://github.com/organicmaps/organicmaps/runs/3273269780?check_suite_focus=true#step:3:1
rtsisyk reviewed 2021-08-08 10:57:47 +00:00
Author
Owner

AppStore, GooglePlay и F-Droid теперь проверяются в Pull Requests при изменении соответствующих директорий:

https://github.com/organicmaps/organicmaps/runs/3273269781?check_suite_focus=true#step:3:1
https://github.com/organicmaps/organicmaps/runs/3273269780?check_suite_focus=true#step:3:1

AppStore, GooglePlay и F-Droid теперь проверяются в Pull Requests при изменении соответствующих директорий: https://github.com/organicmaps/organicmaps/runs/3273269781?check_suite_focus=true#step:3:1 https://github.com/organicmaps/organicmaps/runs/3273269780?check_suite_focus=true#step:3:1
rtsisyk reviewed 2021-08-08 10:59:02 +00:00
Author
Owner

Можем сделать симлинк из android/src/google/play/ в android/metadata. Обрати внимание, что теперь есть android/src/google/play/ и android/src/fdroid/play/ - две копии, чтобы иметь разный текст.

Можем сделать симлинк из `android/src/google/play/` в `android/metadata`. Обрати внимание, что теперь есть `android/src/google/play/` и `android/src/fdroid/play/` - две копии, чтобы иметь разный текст.
rtsisyk reviewed 2021-08-08 11:00:19 +00:00
Author
Owner

Скриншоты вынесены в отдельную репу чтобы не забить здесь всё окончательно. Скриншоты мы меняем редко, поэтому проблемы не вижу.

Скриншоты вынесены в [отдельную репу](https://github.com/organicmaps/screenshots) чтобы не забить здесь всё окончательно. Скриншоты мы меняем редко, поэтому проблемы не вижу.
rtsisyk reviewed 2021-08-08 11:01:07 +00:00
Author
Owner

Я поправил это место. Гугл плей синхронизировал с последними правками F-Droid. Хочу доделать инфраструктуру для работы с метой в гите, а дальше уже можно будет править там как угодно что угодно.

Я поправил это место. Гугл плей синхронизировал с последними правками F-Droid. Хочу доделать инфраструктуру для работы с метой в гите, а дальше уже можно будет править там как угодно что угодно.
rtsisyk reviewed 2021-08-08 11:02:09 +00:00
Author
Owner

Поменял здесь. Для этого и делаем в git всё!

Поменял здесь. Для этого и делаем в git всё!
rtsisyk reviewed 2021-08-08 11:03:16 +00:00
Author
Owner

Я переделал скрипт полностью. Игнорируется только последний "\n". Требуем его наличия в файле, т.к. это наиболее привычное поведение при работе в любом редакторе. При проверке длины не учитываем.

Я переделал скрипт полностью. Игнорируется только последний "\n". Требуем его наличия в файле, т.к. это наиболее привычное поведение при работе в любом редакторе. При проверке длины не учитываем.
biodranik (Migrated from github.com) reviewed 2021-08-08 12:43:58 +00:00
biodranik (Migrated from github.com) commented 2021-08-08 12:43:58 +00:00

Я хорошенько подумал, давай пока сделаем симлинки на все идентичные поля, чтобы не усложнять себе жизнь (кроме whats new, там могут релизы слегка различаться, можно в фдроид вообще чаще фиксы для теста кидать). А как только понадобится форкнуть, просто симлинки заменим файлами.

Я хорошенько подумал, давай пока сделаем симлинки на все идентичные поля, чтобы не усложнять себе жизнь (кроме whats new, там могут релизы слегка различаться, можно в фдроид вообще чаще фиксы для теста кидать). А как только понадобится форкнуть, просто симлинки заменим файлами.
biodranik (Migrated from github.com) reviewed 2021-08-08 12:48:23 +00:00
biodranik (Migrated from github.com) commented 2021-08-08 12:48:23 +00:00

Фейлиться из-за пропущенного конца строки тоже очень странно. Должно работать и так и так. Кейз: я создаю ПР с мобильного гитхаб редактора.

Фейлиться из-за пропущенного конца строки тоже очень странно. Должно работать и так и так. Кейз: я создаю ПР с мобильного гитхаб редактора.
rtsisyk reviewed 2021-08-08 13:32:33 +00:00
Author
Owner

Я могу убрать эту проверку, но мне кажется более правильным форсировать наличие пустой строки вообще во всех файлах. В противном случае постоянно будем иметь левые диффы в зависимости от настроек редактора.

Я могу убрать эту проверку, но мне кажется более правильным форсировать наличие пустой строки вообще во всех файлах. В противном случае постоянно будем иметь левые диффы в зависимости от настроек редактора.
rtsisyk reviewed 2021-08-08 14:29:55 +00:00
Author
Owner

Сделал симлинки из фдроида в гуглплей. Надеюсь, что у них это всё будет работать.

Сделал симлинки из фдроида в гуглплей. Надеюсь, что у них это всё будет работать.
rtsisyk reviewed 2021-08-08 14:30:06 +00:00
Author
Owner

Этот скрипт уничтожен.

Этот скрипт уничтожен.
rtsisyk reviewed 2021-08-08 14:37:07 +00:00
Author
Owner

@biodranik Я закончил с этим патчей в целом. Нюансы следующие:

  1. Заливка скриншотов в AppStore возможна только при наличии черновика билда, который при этом не на ревью. Создать новый черновик пока есть существующий билд на ревью нельзя. По крайне мере в гуи я это не нашел. План - попробовать после прохождения ревью создать черновик и дернуть пайплайн. Пока что он стоит на ручном запуске. В целом думаю тупо интегрировать его в заливку билдов в iOS. Будем заливать билд, обновлять скриншоты, отправлять на ревью.

  2. Скачивания метаданных из гуглплея очень мутное. Оно делает rm -rf на всю папку и всё выкачивает заново. Вместо .jpg почему-то оказываются .png, хотя в реальности там JPEG. Я написал скрипт, учитывающий нюансы, но юзать его надо с осторожностью.

  3. Сделал симлинки из fdroid в gplay. Предлагаю точечно вручную рулить где нужен симлинк, а где будет свой файл.

  4. Скриншоты в андроиде также точечно симликнуты в отдельную репу. Фдроид юзает 4 скрина из README прямо из основного репо. Гугл плей юзает скрины из отдельной репы. Изначально я пытался в build.gradle написать правильную расстановку симлинков, но там это всё делает очень неудобно и я решил проставить симлинки прямо в репе. Может не самое элегантное решение, но зато очень дуракоупорное и точно работающее.


Очень хочется закрыть эти вопросы уже. Планирую еще ребейзнуть заливку iOS поверх этого и потестировать с новым xcode проектом.

@biodranik Я закончил с этим патчей в целом. Нюансы следующие: 1. Заливка скриншотов в AppStore возможна только при наличии черновика билда, который при этом не на ревью. Создать новый черновик пока есть существующий билд на ревью нельзя. По крайне мере в гуи я это не нашел. План - попробовать после прохождения ревью создать черновик и дернуть пайплайн. Пока что он стоит на ручном запуске. В целом думаю тупо интегрировать его в заливку билдов в iOS. Будем заливать билд, обновлять скриншоты, отправлять на ревью. 2. Скачивания метаданных из гуглплея очень мутное. Оно делает rm -rf на всю папку и всё выкачивает заново. Вместо .jpg почему-то оказываются .png, хотя в реальности там JPEG. Я написал скрипт, учитывающий нюансы, но юзать его надо с осторожностью. 3. Сделал симлинки из fdroid в gplay. Предлагаю точечно вручную рулить где нужен симлинк, а где будет свой файл. 4. Скриншоты в андроиде также точечно симликнуты в отдельную репу. Фдроид юзает 4 скрина из README прямо из основного репо. Гугл плей юзает скрины из отдельной репы. Изначально я пытался в build.gradle написать правильную расстановку симлинков, но там это всё делает очень неудобно и я решил проставить симлинки прямо в репе. Может не самое элегантное решение, но зато очень дуракоупорное и точно работающее. --- Очень хочется закрыть эти вопросы уже. Планирую еще ребейзнуть заливку iOS поверх этого и потестировать с новым xcode проектом.
biodranik (Migrated from github.com) reviewed 2021-08-09 06:34:46 +00:00
biodranik (Migrated from github.com) commented 2021-08-09 06:34:46 +00:00
  1. Норм.
  2. Норм но надо зарепортить баг или пофиксить в апстриме.
  3. Норм.
  4. Норм.
1. Норм. 2. Норм но надо зарепортить баг или пофиксить в апстриме. 3. Норм. 4. Норм.
rtsisyk reviewed 2021-08-09 06:41:04 +00:00
Author
Owner
баг https://github.com/Triple-T/gradle-play-publisher/issues/992
biodranik (Migrated from github.com) requested changes 2021-08-11 20:47:38 +00:00
biodranik (Migrated from github.com) left a comment

В целом ок, но не понятно откуда вылезли изменения в twine.

В целом ок, но не понятно откуда вылезли изменения в twine.
biodranik (Migrated from github.com) commented 2021-08-11 20:25:58 +00:00

Если разрешать ручной триггер, тогда можно будет выбрать явно бранч для запуска.

Если разрешать ручной триггер, тогда можно будет выбрать явно бранч для запуска.
biodranik (Migrated from github.com) commented 2021-08-11 20:26:47 +00:00

С ручным триггером это не нужно.

С ручным триггером это не нужно.
biodranik (Migrated from github.com) commented 2021-08-11 20:27:11 +00:00

Почему закомменчено?

Почему закомменчено?
biodranik (Migrated from github.com) commented 2021-08-11 20:28:57 +00:00

Зачем так долго?

Зачем так долго?
@ -0,0 +15,4 @@
shell: bash
run: git submodule update --init --recursive --jobs=$(($(nproc) * 2))
- name: Checkout private keys
biodranik (Migrated from github.com) commented 2021-08-11 20:28:04 +00:00

Выше уже есть этот шаг.

Выше уже есть этот шаг.
biodranik (Migrated from github.com) commented 2021-08-11 20:29:32 +00:00

Убрать перед мержем.

Убрать перед мержем.
biodranik (Migrated from github.com) commented 2021-08-11 20:30:31 +00:00

Так долго?

Так долго?
biodranik (Migrated from github.com) commented 2021-08-11 20:30:52 +00:00

Если работает, убирай )

Если работает, убирай )
@ -0,0 +3,4 @@
• Contour lines, elevation profiles, peaks and slopes
• Turn-by-turn walking, cycling and car navigation with voice guidance
• Fast offline search on the map and bookmarks
• Dark mode to protect your eyes
biodranik (Migrated from github.com) commented 2021-08-11 20:36:19 +00:00

• Detailed offline maps with places that don't exist on other maps, thanks to OpenStreetMap
• Cycling routes, hiking trails, and walking paths
• Contour lines, elevation profiles, peaks, and slopes
• Turn-by-turn walking, cycling, and car navigation with voice guidance
• Fast offline search on the map
• Bookmarks export and import in KML/KMZ formats (GPX is planned)
• Dark Mode to protect your eyes
• Countries and regions don't take a lot of space
• Free and open-source

• Detailed offline maps with places that don't exist on other maps, thanks to OpenStreetMap • Cycling routes, hiking trails, and walking paths • Contour lines, elevation profiles, peaks, and slopes • Turn-by-turn walking, cycling, and car navigation with voice guidance • Fast offline search on the map • Bookmarks export and import in KML/KMZ formats (GPX is planned) • Dark Mode to protect your eyes • Countries and regions don't take a lot of space • Free and open-source
@ -0,0 +23,4 @@
• No push notifications
• No crapware
• N̶o̶ ̶p̶e̶s̶t̶i̶c̶i̶d̶e̶s̶
• Purely organic
biodranik (Migrated from github.com) commented 2021-08-11 20:36:59 +00:00

Вставь зачёркивание если работает.

Вставь зачёркивание если работает.
biodranik (Migrated from github.com) commented 2021-08-11 20:37:16 +00:00
  • No pesticides Purely organic! (только не отсюда, я в другом месте копировал)
- ~~No pesticides~~ Purely organic! (только не отсюда, я в другом месте копировал)
@ -0,0 +48,4 @@
### Uploading a new version to Google Play
Use [GitHub Actions](../.github/workflows/android-release.yaml).
biodranik (Migrated from github.com) commented 2021-08-11 20:38:35 +00:00

Эти ссылки рабочие?

Эти ссылки рабочие?
@ -0,0 +1,37 @@
• Detailed offline maps with places that don't exist on other maps
• Cycling routes, hiking trails, and walking paths
• Contour lines, elevation profiles, peaks, and slopes
• Turn-by-turn walking, cycling, and car navigation with voice guidance
biodranik (Migrated from github.com) commented 2021-08-11 20:39:38 +00:00

Вставь то же что и выше, тут car замени на CarPlay

Вставь то же что и выше, тут car замени на CarPlay
biodranik (Migrated from github.com) commented 2021-08-11 20:42:52 +00:00

Лучше писать полный путь к keys, скрипт можно ранить из любой диры.

Лучше писать полный путь к keys, скрипт можно ранить из любой диры.
biodranik (Migrated from github.com) commented 2021-08-11 20:43:27 +00:00

И тут лучше логать полный путь.

И тут лучше логать полный путь.
@ -0,0 +5,4 @@
set -euo pipefail
cd "$(dirname "$0")/../../android"
biodranik (Migrated from github.com) commented 2021-08-11 20:41:03 +00:00

Это ненадёжно, лучше SCRIPT_DIR="$( cd "$(dirname "$0")" && pwd -P )"

Это ненадёжно, лучше `SCRIPT_DIR="$( cd "$(dirname "$0")" && pwd -P )"`
@ -0,0 +46,4 @@
}
check_keys
check_screenshots
biodranik (Migrated from github.com) commented 2021-08-11 20:44:24 +00:00

Зачем это для загрузки данных из стора?

Зачем это для загрузки данных из стора?
biodranik (Migrated from github.com) commented 2021-08-11 20:45:19 +00:00

Почему закомменчено?

Почему закомменчено?
rtsisyk reviewed 2021-08-12 08:21:40 +00:00
Author
Owner

Список допустимых бранчей для деплоя в бету/прод всё равно ограничен в GitHub Environments. Я уберу этот хак, он только для тестирования нужен был.

Список допустимых бранчей для деплоя в бету/прод всё равно ограничен в GitHub Environments. Я уберу этот хак, он только для тестирования нужен был.
rtsisyk reviewed 2021-08-12 08:21:51 +00:00
Author
Owner

Уберу это.

Уберу это.
rtsisyk reviewed 2021-08-12 08:22:07 +00:00
Author
Owner

Не срабатывало потому что. Это всё временное, я уберу из финального патча.

Не срабатывало потому что. Это всё временное, я уберу из финального патча.
rtsisyk reviewed 2021-08-12 08:22:14 +00:00
@ -0,0 +15,4 @@
shell: bash
run: git submodule update --init --recursive --jobs=$(($(nproc) * 2))
- name: Checkout private keys
Author
Owner

ок

ок
rtsisyk reviewed 2021-08-12 08:22:24 +00:00
Author
Owner

Можем меньше поставить, не проблема.

Можем меньше поставить, не проблема.
rtsisyk reviewed 2021-08-12 08:23:18 +00:00
@ -0,0 +5,4 @@
set -euo pipefail
cd "$(dirname "$0")/../../android"
Author
Owner

Так cd "$(dirname "$0")/../../android" тоже самое, что cd "$(dirname "$0")" внутри. Я просто делаю chdir в эту папку, сама переменная мне не нужна.

Так `cd "$(dirname "$0")/../../android"` тоже самое, что `cd "$(dirname "$0")"` внутри. Я просто делаю chdir в эту папку, сама переменная мне не нужна.
rtsisyk reviewed 2021-08-12 08:23:35 +00:00
Author
Owner

Там выше chdir стоит же, чтобы везде эти пути не прописывать.

Там выше chdir стоит же, чтобы везде эти пути не прописывать.
rtsisyk reviewed 2021-08-12 08:24:04 +00:00
@ -0,0 +46,4 @@
}
check_keys
check_screenshots
Author
Owner

Оно сразу в репу со скринами подпихивает скачанные файлы в нужные папки.

Оно сразу в репу со скринами подпихивает скачанные файлы в нужные папки.
rtsisyk reviewed 2021-08-12 08:24:20 +00:00
Author
Owner

Артефакт, удалю.

Артефакт, удалю.
rtsisyk reviewed 2021-08-12 08:28:15 +00:00
Author
Owner

Удалил.

Удалил.
rtsisyk reviewed 2021-08-12 08:28:39 +00:00
Author
Owner

Поставил 5 минут. Меньше рисковано, все же там весьма немаленкие скриншоты.

Поставил 5 минут. Меньше рисковано, все же там весьма немаленкие скриншоты.
Author
Owner

Twine попал случайно, забыл обновить сабмодуль и он добавился на git add.

#1085

Перепушал.

Twine попал случайно, забыл обновить сабмодуль и он добавился на git add. #1085 Перепушал.
biodranik (Migrated from github.com) reviewed 2021-08-12 12:06:38 +00:00
biodranik (Migrated from github.com) commented 2021-08-12 12:06:38 +00:00

Для отладки полезно, т.к. не понятно, в какой папке что должно лежать в случае ошибки.

Для отладки полезно, т.к. не понятно, в какой папке что должно лежать в случае ошибки.
biodranik (Migrated from github.com) reviewed 2021-08-12 12:12:22 +00:00
biodranik (Migrated from github.com) left a comment

Многие комменты без ответа

Многие комменты без ответа
@ -0,0 +26,4 @@
with open(path, 'r') as f:
text = f.read()
if text[-1] == os.linesep:
text = text[:-1]
biodranik (Migrated from github.com) commented 2021-08-12 12:12:05 +00:00

А в сторы такие переносы строк сверх лимита стрипаются и тоже не учитываются? Проверял на макс.длине текста?

А в сторы такие переносы строк сверх лимита стрипаются и тоже не учитываются? Проверял на макс.длине текста?
rtsisyk reviewed 2021-08-12 20:48:53 +00:00
Author
Owner

Давай по android/src/google/play/ триггерить. Ну то есть когда поменялась мета для гугл плея.

Давай по android/src/google/play/ триггерить. Ну то есть когда поменялась мета для гугл плея.
rtsisyk reviewed 2021-08-12 20:49:42 +00:00
Author
Owner

Хорошо, поменя здесь.

Хорошо, поменя здесь.
rtsisyk reviewed 2021-08-12 20:49:50 +00:00
Author
Owner

Хорошо, поменяю здесь.

Хорошо, поменяю здесь.
rtsisyk reviewed 2021-08-12 20:50:10 +00:00
@ -0,0 +26,4 @@
with open(path, 'r') as f:
text = f.read()
if text[-1] == os.linesep:
text = text[:-1]
Author
Owner

Надо попробовать.

Надо попробовать.
Author
Owner

Вроде везде ответил. Поправлю оставшуюся мелочь. Посмотри что еще осталось.

Вроде везде ответил. Поправлю оставшуюся мелочь. Посмотри что еще осталось.
rtsisyk reviewed 2021-08-14 05:39:12 +00:00
Author
Owner

Добавил $PWD в вывод ошибки. chdir убирать не хочу, неудобно в каждой строчке полный путь писать.

Добавил $PWD в вывод ошибки. chdir убирать не хочу, неудобно в каждой строчке полный путь писать.
rtsisyk reviewed 2021-08-14 05:42:58 +00:00
@ -0,0 +26,4 @@
with open(path, 'r') as f:
text = f.read()
if text[-1] == os.linesep:
text = text[:-1]
Author
Owner

title.txt "Organic Maps Offline Hike Bike Trails & Navigation" как раз 50 символов + "\n" и всё заливается.

b02ced7f27/common/utils/src/main/kotlin/com/github/triplet/gradle/common/utils/Io.kt (L55-L56)

title.txt "Organic Maps Offline Hike Bike Trails & Navigation" как раз 50 символов + "\n" и всё заливается. https://github.com/Triple-T/gradle-play-publisher/blob/b02ced7f277a68ddfa8bd83b12508ad5beed3c14/common/utils/src/main/kotlin/com/github/triplet/gradle/common/utils/Io.kt#L55-L56
rtsisyk reviewed 2021-08-14 05:43:11 +00:00
@ -0,0 +26,4 @@
with open(path, 'r') as f:
text = f.read()
if text[-1] == os.linesep:
text = text[:-1]
Author
Owner

Они сами стрипают там всё.

Они сами стрипают там всё.
Author
Owner

Discussed offline. Please feel free to improve in master.

Discussed offline. Please feel free to improve in master.
This repo is archived. You cannot comment on pull requests.
No reviewers
No labels
Accessibility
Accessibility
Address
Address
Android
Android
Android Auto
Android Auto
Android Automotive (AAOS)
Android Automotive (AAOS)
API
API
AppGallery
AppGallery
AppStore
AppStore
Battery and Performance
Battery and Performance
Blocker
Blocker
Bookmarks and Tracks
Bookmarks and Tracks
Borders
Borders
Bug
Bug
Build
Build
CarPlay
CarPlay
Classificator
Classificator
Community
Community
Core
Core
CrashReports
CrashReports
Cycling
Cycling
Desktop
Desktop
DevEx
DevEx
DevOps
DevOps
dev_sandbox
dev_sandbox
Directions
Directions
Documentation
Documentation
Downloader
Downloader
Drape
Drape
Driving
Driving
Duplicate
Duplicate
Editor
Editor
Elevation
Elevation
Enhancement
Enhancement
Epic
Epic
External Map Datasets
External Map Datasets
F-Droid
F-Droid
Fonts
Fonts
Frequently User Reported
Frequently User Reported
Fund
Fund
Generator
Generator
Good first issue
Good first issue
Google Play
Google Play
GPS
GPS
GSoC
GSoC
iCloud
iCloud
Icons
Icons
iOS
iOS
Legal
Legal
Linux Desktop
Linux Desktop
Linux packaging
Linux packaging
Linux Phone
Linux Phone
Mac OS
Mac OS
Map Data
Map Data
Metro
Metro
Navigation
Navigation
Need Feedback
Need Feedback
Night Mode
Night Mode
NLnet 2024-06-281
NLnet 2024-06-281
No Feature Parity
No Feature Parity
Opening Hours
Opening Hours
Outdoors
Outdoors
POI Info
POI Info
Privacy
Privacy
Public Transport
Public Transport
Raw Idea
Raw Idea
Refactoring
Refactoring
Regional
Regional
Regression
Regression
Releases
Releases
RoboTest
RoboTest
Route Planning
Route Planning
Routing
Routing
Ruler
Ruler
Search
Search
Security
Security
Styles
Styles
Tests
Tests
Track Recording
Track Recording
Translations
Translations
TTS
TTS
UI
UI
UX
UX
Walk Navigation
Walk Navigation
Watches
Watches
Web
Web
Wikipedia
Wikipedia
Windows
Windows
Won't fix
Won't fix
World Map
World Map
No milestone
No project
No assignees
2 participants
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: organicmaps/organicmaps-tmp#1035
No description provided.