Skip to content
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 changes from release 2.21 finalization #1212

Merged
merged 8 commits into from
Aug 30, 2024
Merged

Merge changes from release 2.21 finalization #1212

merged 8 commits into from
Aug 30, 2024

Conversation

mokagio
Copy link
Contributor

@mokagio mokagio commented Aug 30, 2024

Notice it's all localizations, the build number, and... a bunch of new automation for the download and upload of localized metadata.

Something must have changed in App Store Connect since the last submission of Simplenote macOS, as I was not able to submit the app for review the first time I tried due to missing localized metadata, something that (as the lack of automation shows) we never run into before.

Anyway... I added just enough automation to get it done. More to come as part of the release automation modernization for Simplenote, internal ref paaHJt-6Zx-p2).

@Automattic/apps-infrastructure I'd appreciate a look, even though the fact that everything worked is already good proof of the validity of the changes. Thanks!

See also:

@mokagio mokagio added the Releases Release related stuff label Aug 30, 2024
@mokagio mokagio added this to the Future milestone Aug 30, 2024
@mokagio mokagio enabled auto-merge August 30, 2024 12:17
@dangermattic
Copy link
Collaborator

1 Message
📖 This PR has the Releases label: some checks will be skipped.

Generated by 🚫 Danger

Copy link
Contributor

@jleandroperez jleandroperez left a comment

Choose a reason for hiding this comment

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

:shipit:

@mokagio mokagio merged commit f686d1b into trunk Aug 30, 2024
8 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Releases Release related stuff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants