Update Ruby Gems for the latest release-toolkit version

irfano commented 17 hours ago

Description

This only updates gems. The purpose is using the latest release-toolkit version.

Testing information

It seems safe to me not to test anything, but let me know if I missed anything, @iangmaia.

[x] I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

Reviewer (or Author, in the case of optional code reviews):

Please make sure these conditions are met before approving the PR, or request changes if the PR needs improvement:

[ ] The PR is small and has a clear, single focus, or a valid explanation is provided in the description. If needed, please request to split it into smaller PRs.
[ ] Ensure Adequate Unit Test Coverage: The changes are reasonably covered by unit tests or an explanation is provided in the PR description.
[ ] Manual Testing: The author listed all the tests they ran, including smoke tests when needed (e.g., for refactorings). The reviewer confirmed that the PR works as expected on all devices (phone/tablet) and no regressions are added.

wpmobilebot commented 6 hours ago

Version 21.2 has now entered code-freeze, so the milestone of this PR has been updated to 21.3.

hafizrahman commented 6 hours ago

After the PR was approved, I saw a build error that says `Podfile.lock was changed by bundle exec pod install. Please run bundle exec pod install locally with a clean cache and commit the result."

I did as it said and pushed in https://github.com/woocommerce/woocommerce-ios/pull/14487/commits/3ada1876f710c2a55b54afbfb434ec83f94db848

wpmobilebot commented 6 hours ago

📲 You can test the changes from this Pull Request in WooCommerce iOS by scanning the QR code below to install the corresponding build.

	App Name	WooCommerce iOS
	Build Number	pr14487-3ada187
	Version	21.1
	Bundle ID	com.automattic.alpha.woocommerce
	Commit	3ada1876f710c2a55b54afbfb434ec83f94db848
	App Center Build	WooCommerce - Prototype Builds #11720

Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.

dangermattic commented 2 hours ago

	1 Error
:no_entry_sign:	This PR is tagged with `status: do not merge` label(s).

Generated by :no_entry_sign: Danger

hafizrahman commented 2 hours ago

This is good to go but I'm setting "do not merge" for now. We want to put it into release 21.2 so we can test screenshots generation in release. I'm not sure what's the next steps are here as we're just in code freeze @selanthiraiyan

selanthiraiyan commented 1 hour ago

Thanks, @irfano for working on this!

This is good to go but I'm setting "do not merge" for now. We want to put it into release 21.2 so we can test screenshots generation in release. I'm not sure what's the next steps are here as we're just in code freeze @selanthiraiyan

@hafizrahman This PR and the next one here https://github.com/woocommerce/woocommerce-ios/pull/14492 look good to me to merge as beta fixes into the release branch. Will that be all, or do we need to make further changes?

woocommerce / woocommerce-ios