Skip to content

Merge 18.9 code freeze into develop - #17672

Merged
jkmassel merged 12 commits into
developfrom
merge/18.9-code-freeze-into-develop
Dec 16, 2021
Merged

Merge 18.9 code freeze into develop#17672
jkmassel merged 12 commits into
developfrom
merge/18.9-code-freeze-into-develop

Conversation

@jkmassel

Copy link
Copy Markdown
Contributor

Merges the 18.9 code freeze into develop, including:

  • Version number bumps
  • Release toolkit update
  • Fastlane + Fastlane Plugin updates
  • Production Pods
  • Translation pipeline fixes

@jkmassel jkmassel added this to the 18.9 ❄️ milestone Dec 14, 2021
@jkmassel
jkmassel requested review from a team and AliSoftware December 14, 2021 02:49
@jkmassel jkmassel self-assigned this Dec 14, 2021
@peril-wordpress-mobile

peril-wordpress-mobile Bot commented Dec 14, 2021

Copy link
Copy Markdown

You can trigger an installable build for these changes by visiting CircleCI here.

Comment thread fastlane/Fastfile Outdated
Comment thread fastlane/Fastfile
output_dir: en_lproj
)

# @TODO: Remove once we migrated the codebase of the widgets to use the app bundle for .strings

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AliSoftware – I needed to remove these in order to complete the code freeze – I believe these were resolve as part of #17630 and #17636 , but LMK if anything doesn't look right?

@AliSoftware AliSoftware Dec 14, 2021

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's correct and I should have removed those as part of those PRs, thanks for taking care of this.

There might still be an issue waiting to happen during new_beta_release and finalize_release when those will call the scripts downloading the translations from GlotPress and those scripts (which are still yet to be migrated) might still be trying to re-dispatch the keys to those widget targets based on those (now deleted) Base.lproj keys. Until that side of the migration of our scripts is over we might need to patch the extract-framework-translations.swift scripts (or even stop calling that script altogether by removing these lines) to stop doing that for those targets until I finalize the related work.

@peril-wordpress-mobile

peril-wordpress-mobile Bot commented Dec 14, 2021

Copy link
Copy Markdown

You can trigger optional UI/connected tests for these changes by visiting CircleCI here.

@jkmassel
jkmassel merged commit 857ae99 into develop Dec 16, 2021
@jkmassel
jkmassel deleted the merge/18.9-code-freeze-into-develop branch December 16, 2021 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants