Skip to content

Conversation

@mokagio
Copy link
Contributor

@mokagio mokagio commented Sep 8, 2021

Please forgive the long commit history. I was curious to try my msgid fix, which is merged in the release toolkit develop but hasn't been release yet, in another codebase but pointed to trunk instead of develop and, because update_appstore_strings pushes, I didn't have a chance to rebase the mistake away. FWIW, 69e4335 and confirm 7b3ecb5 one more time that the fix is legit.

@peril-wordpress-mobile
Copy link

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

faraday (>= 0.9)
sawyer (~> 0.8.0, >= 0.5.3)
oj (3.13.1)
oj (3.13.4)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

These gems patch version changes are a byproduct of my dance with the release toolkit version. They snuck in there because I had to go through two versions, so instead of git revert two commits I manually rolled back the Pluginfile and run bundle update fastlane-plugin-wpmreleasetoolkit.

I don't think they present any issue, so I decided not to spend any extra time to amend them.

@peril-wordpress-mobile
Copy link

Messages
📖 This PR has the 'Releases' label: some checks will be skipped.

Generated by 🚫 dangerJS

@mokagio mokagio requested a review from a team September 8, 2021 04:50
@mokagio mokagio marked this pull request as ready for review September 8, 2021 04:50
@mokagio mokagio enabled auto-merge September 8, 2021 04:50
@mokagio mokagio added this to the 18.2 ❄️ milestone Sep 8, 2021
@mokagio mokagio merged commit 6e1384d into develop Sep 8, 2021
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