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

Release/3.14.2 #1553

Merged
merged 2 commits into from Apr 29, 2022
Merged

Release/3.14.2 #1553

merged 2 commits into from Apr 29, 2022

Conversation

joshdholtz
Copy link
Contributor

@joshdholtz joshdholtz requested a review from a team April 29, 2022 19:20
@joshdholtz joshdholtz changed the base branch from main to release/3.x.x April 29, 2022 19:21
@joshdholtz
Copy link
Contributor Author

@NachoSoto The Carthage integration tests are failing because of reasons that I don't think are super important but 🤷‍♂️ Is this still okay to merge and release? 🙃

@NachoSoto
Copy link
Contributor

What's the failure?

@joshdholtz
Copy link
Contributor Author

@NachoSoto It looks like its checking out 4.3.0 so Magic Weather is failing because its built with v3 stuff on this branch 😛

@NachoSoto
Copy link
Contributor

NachoSoto commented Apr 29, 2022

Looking at that now it seems wrong. The new version correctly replaces main with the right commit.
But in 3.x this seems wrong, it's replacing CARTHAGE_INTEGRATION_TESTS_GIT_COMMIT but it's not there?
I would recommend fixing it, maybe Carthage tests haven't worked for a while in 3.x releases?

@joshdholtz
Copy link
Contributor Author

@NachoSoto Looks like we gucci now!

Copy link
Contributor

@NachoSoto NachoSoto left a comment

Choose a reason for hiding this comment

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

Yay!

git "./purchases-root" "CARTHAGE_INTEGRATION_TESTS_GIT_COMMIT"
Copy link
Contributor

Choose a reason for hiding this comment

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

❤️

@joshdholtz joshdholtz merged commit afc44aa into release/3.x.x Apr 29, 2022
@joshdholtz joshdholtz deleted the release/3.14.2 branch April 29, 2022 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants