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

chore: update main-swift branch from main branch #522

Open
wants to merge 8 commits into
base: main-swift
Choose a base branch
from

Conversation

iHackSubhodip
Copy link
Collaborator

What's changing?

  • update main-swift branch from main branch

talipboke and others added 4 commits November 7, 2024 08:07
Previously, when subclassing GMSMarker with a custom iconView, the iconView would not display on the map. This update enables custom iconViews to render correctly when using GMSMarker subclasses.
* chore(main): release 6.1.0

* chore(release-please--branches--main): Patch to release branch 6.1.0 (#509)

---------

Co-authored-by: Subhodip Banerjee <[email protected]>
@iHackSubhodip iHackSubhodip requested a review from a team as a code owner January 21, 2025 15:28
@@ -41,7 +41,7 @@ jobs:
- name: Set Xcode version
uses: maxim-lobanov/setup-xcode@v1
with:
xcode-version: latest-stable
xcode-version: "16.1"
Copy link
Collaborator Author

@iHackSubhodip iHackSubhodip Jan 21, 2025

Choose a reason for hiding this comment

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

[self-review]

What?

  • This change should not be in as part of this PR. As, I just want to update main-swift branch from main branch.
  • This should be latest Xcode 16.2[xcode-version: latest-stable]. With the latest version the CI is failing. Attaching the CI job for reference. [https://github.com/googlemaps/google-maps-ios-utils/actions/runs/12890243952/job/35939379429]

Why? [Failure Reason]

  • The CI is failing with latest Xcode.
  • with Xcode 16.2 version, I don't see any required simulators in the Github CI.

How?

  • Changing it to the previous Xcode version, making the CI success.
  • The correct fix should be making the required simulator available for Xcode 16.2

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.

5 participants