mapbox / mapbox/MapboxStatic.swift

Build universal framework target for Carthage

Open
#117 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

build
Dominant language
Swift
Stars
192
Forks
31
PR merge metrics
No merged PRs in 30d

Description

MapboxStatic.xcodeproj currently has four separate framework targets, one for each platform, plus corresponding test targets and schemes. To streamline development both locally and downstream, we should combine them into a single framework target and test target. Downstream, only Carthage uses the Xcode project; it has supported universal frameworks since v0.9. The CircleCI configuration needs to be updated to refer to the new combined scheme.

/ref mapbox/turf-swift#197 mapbox/mapbox-directions-swift#730

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Inspect MapboxStatic.xcodeproj to understand the four framework targets, their test targets, and schemes. Then review the CircleCI configuration and Carthage's universal framework support. Done means one combined framework target and test target are used, with CircleCI referring to the combined scheme.

Written by the indexing model from the issue text.

Assessment

Tech stack
swift
Domain
build-system, ci-cd
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.