Fixes for the xcodeproj file

[jasdeepsaini]

Fixes to get the project and tests to build:

• Deleted the missing MapTo and MapToTests files.
• Change the deployment target for the CombineExtTests target to 13.0 so the latest version of CombineSchedulers can be used.
• Removed CombineSchedulers from the CombineExt target and added it to CombineExtTests. This allows the project to build since CombineExt supports iOS 10.0 and CombineSchedulers requires iOS 13.
• Updated the CombineSchedulers package to 0.5.3 to match the Package file.
• The Package.resolved file automatically got updated to v2 since I am using Xcode 13.

Project cleanup

• Deleted the missing Carthage and CHANGELOG.MD files.

Missing library and test files added to the project:

• MapToValue.swift
• MapToValueTests.swift
• MapToResult.swift
• MapToResultTests.swift
• Enumerated.swift
• EnumeratedTests.swift

[jasdeepsaini]

I think the .DS_Store file should be added to .gitignore, but I didn't make that change in this PR. Is there a reason to commit this file?

[freak4pc]

Would be great to add it ! Thanks

[jasdeepsaini]

Done

[freak4pc]

I'll tryto find some time to review this - I'll start by saying I'd prefer keeping the deployment target low (iOS 10 or 11), because thjere could be some older codebases that don't necessarily use Combine all over the place but can still benefit from using CombineExt in some places that are guarded by availability clauses.

[jasdeepsaini]

I'll tryto find some time to review this - I'll start by saying I'd prefer keeping the deployment target low (iOS 10 or 11), because thjere could be some older codebases that don't necessarily use Combine all over the place but can still benefit from using CombineExt in some places that are guarded by availability clauses.

The deployment target is an issue if you want to use the latest version of Combine-Schedulers which requires iOS 13. With the lower deployment target, the library and unit tests won't build. The Package.swift file has a minimum iOS version of 13 so older projects won't be able to pull in this package.

[freak4pc]

I'll have to look. I'm not sure if there's a way to give a different deploymetn target to the test target.

[freak4pc]

Anyways if that's the case we can leave the old version of combine-schedulers, It's just a test utility so I'm not sure it matters too much.

[jasdeepsaini]

Anyways if that's the case we can leave the old version of combine-schedulers, It's just a test utility so I'm not sure it matters too much.

I reverted the deployment target of CombineExt to iOS 10.0 and changed the deployment target of CombineExtTests to iOS 13.0. Then I moved the CombineSchedulers package from CombineExt to CombineExtTests.

This should do what you wanted while allowing tests to be run if CombineExt is modified in Xcode instead of the command line.

[freak4pc]

LGTM. Let's just fix the conflicts :)
(Or add DS_Store to .gitignore which would be even better)

[jasdeepsaini]

LGTM. Let's just fix the conflicts :)
(Or add DS_Store to .gitignore which would be even better)

I added .DS_Store to .gitignore.

[codecov]

Codecov Report

Merging #133 (1cd51ba) into main (5c17b57) will not change coverage.
The diff coverage is n/a.

❗ Current head 1cd51ba differs from pull request most recent head 72e610b. Consider uploading reports for the commit 72e610b to get more accurate results

@@           Coverage Diff           @@
##             main     #133   +/-   ##
=======================================
  Coverage   95.51%   95.51%           
=======================================
  Files          70       70           
  Lines        4166     4166           
=======================================
  Hits         3979     3979           
  Misses        187      187           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5c17b57...72e610b. Read the comment docs.