-
Notifications
You must be signed in to change notification settings - Fork 301
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merge 2.34 code freeze #1692
Merged
Merged
Merge 2.34 code freeze #1692
Changes from 17 commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
5138a01
[Bug] If a deeplink is malformed or missing data we show auth screen.
notandyvee 7e68496
Show loading dialog when loading magic link.
notandyvee 19f21bc
Bump version name and build code
wpmobilebot 1b99c64
Update draft release notes for 2.34.
wpmobilebot cc63d77
Release Notes: add new section for next version (2.35)
wpmobilebot 0ad4fbb
Update strings file for translation automation
wpmobilebot 9b72379
Update release notes source with placeholder copy for 2.34
mokagio b88bf41
Merge remote-tracking branch 'origin/trunk' into andy/firebird-issue-81
mokagio bb90134
Update release notes source with placeholder copy for 2.34 (#1691)
mokagio bfea8bc
Use `RELEASE_VERSION` instead of `BUILDKITE_RELEASE_VERSION`
mokagio 6fce8c8
Update `.configure` using latest toolkit version
mokagio 080269b
Update `PlayStoreStrings.pot` for 2.34
wpmobilebot 30013e5
Fix `BUILDKITE_ORGANIZATION` defined as `BUILDKITE_ORG`
mokagio c1c0b2d
Add missing `version` and `build_code` to `build_for_distribution`
mokagio 18aba59
Account or different globbing behavior in Bash vs Zsh
mokagio 30ef95b
Specify `-maxdepth` before `-name`
mokagio 057c1b7
Make all release Buildkite command scripts executable
mokagio 4ed1be7
Fix incorrect APK path in upload script
mokagio 98e9ccc
Use Fastlane to decrypt secrets in prototype build step
mokagio 3b01a75
Use full release name when creating GitHub release
mokagio bd79e42
[Bug] If a deeplink is malformed or missing data we show auth screen.…
notandyvee File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,14 +1,12 @@ | ||
#!/bin/bash -eu | ||
|
||
# We expect BUILDKITE_RELEASE_VERSION to be as an environment variable, e.g. by the automation that triggers the build on Buildkite. | ||
# It must use the `BUILDKITE_` prefix to be passed to the agent due to how `hostmgr` works, in case this runs on a Mac agents. | ||
if [[ -z "${BUILDKITE_RELEASE_VERSION}" ]]; then | ||
echo "BUILDKITE_RELEASE_VERSION is not set." | ||
if [[ -z "${RELEASE_VERSION}" ]]; then | ||
echo "RELEASE_VERSION is not set." | ||
exit 1 | ||
fi | ||
|
||
# Buildkite, by default, checks out a specific commit. | ||
# For many release actions, we need to be on a release branch instead. | ||
BRANCH_NAME="release/${BUILDKITE_RELEASE_VERSION}" | ||
BRANCH_NAME="release/${RELEASE_VERSION}" | ||
git fetch origin "$BRANCH_NAME" | ||
git checkout "$BRANCH_NAME" |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Empty file.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,27 +1,35 @@ | ||
{ | ||
"project_name": "Simplenote-Android", | ||
"branch": "trunk", | ||
"pinned_hash": "9b6326550a87fe0b6423a9e12f79ca92be8c9cff", | ||
"pinned_hash": "2c5509d08ff9b9620381f4dc602fc2e5edcfcb40", | ||
"files_to_copy": [ | ||
{ | ||
"file": "android/simplenote/Simplenote/gradle.properties", | ||
"destination": "Simplenote/gradle.properties" | ||
"destination": "Simplenote/gradle.properties", | ||
"encrypt": true | ||
}, | ||
{ | ||
"file": "android/simplenote/sentry.properties", | ||
"destination": "sentry.properties" | ||
"destination": "sentry.properties", | ||
"encrypt": true | ||
}, | ||
{ | ||
"file": "android/automattic.jks", | ||
"destination": ".configure-files/release.jks" | ||
"destination": ".configure-files/release.jks", | ||
"encrypt": true | ||
}, | ||
{ | ||
"file": "android/debug.keystore", | ||
"destination": ".configure-files/debug_a8c.keystore" | ||
"destination": ".configure-files/debug_a8c.keystore", | ||
"encrypt": true | ||
}, | ||
{ | ||
"file": "android/Simplenote/google-upload-credentials.json", | ||
"destination": ".configure-files/google-upload-credentials.json" | ||
"destination": ".configure-files/google-upload-credentials.json", | ||
"encrypt": true | ||
} | ||
], | ||
"file_dependencies": [ | ||
|
||
] | ||
} | ||
} |
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,7 @@ | ||
2.35 | ||
----- | ||
|
||
|
||
2.34 | ||
----- | ||
|
||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
Under the hood improvements |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,2 @@ | ||
versionName=2.33 | ||
versionCode=169 | ||
versionName=2.34-rc-1 | ||
versionCode=170 |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I double-checked and this
checkout-release-branch.sh
script seems to only be called bycomplete-code-freeze.yml
, from a step that runs ontumblr-metal
.So indeed there's no risk of the env var being filtered out by
hostmgr
when the script to run is passed from Mac host to VM here, because this will run ontumblr-metal
and not Mac agents, sohostmgr
is not even involved in the first place. 👍