tuist / XcodeProj

πŸ“ Read, update and write your Xcode projects
https://xcodeproj.tuist.io
MIT License
2.03k stars 309 forks source link

Add `.extensionKitExtension` as the new `PBXProductType` #691

Closed mtj0928 closed 2 years ago

mtj0928 commented 2 years ago

Resolves: https://github.com/tuist/XcodeProj/issues/687

Short description πŸ“

From Xcode14 beta 1, com.apple.product-type.extensionkit-extension was introduced as the new productType. The new productType is used for a new app extension like AppIntents Extension.

Solution πŸ“¦

Added .extensionKitExtension as the new case of PBXProductType in this PR, to support the new productType.

Implementation πŸ‘©β€πŸ’»πŸ‘¨β€πŸ’»

kwridan commented 2 years ago

@all-contributors add @mtj0928 for code

allcontributors[bot] commented 2 years ago

@kwridan

I've put up a pull request to add @mtj0928! :tada:

mtj0928 commented 2 years ago

@kwridan Thank you for your comment!!

I actually considered the name at first , but I also felt it was a bit odd. Would you like me to change the name to extensionKitExtension?

freddi-kit commented 2 years ago

I vote to rename as extensionKitExtension to make naming consistently to others (appExtension, tvExtension)

kwridan commented 2 years ago

cc: @tuist/core - any preference to the naming convention - I could see either option working, consistency vs oddness πŸ˜…

danieleformichelli commented 2 years ago

Slightly in favor of extensionKitExtension to make clear it's an extension, but no strong opinion

fortmarek commented 2 years ago

I'm ok with merging this but I am also leaning to use the more consistent extensionKitExtension. While it does sound weird, I prefer it over having to deal with it down the road if we need to differentiate between different types for extensionKit.

kwridan commented 2 years ago

@mtj0928 can we trouble you to rename to .extensionKitExtension?

The points raised by @fortmarek are valid, while the name is odd, it is consistent with other product types and the underlying Xcode types.

codecov-commenter commented 2 years ago

Codecov Report

Merging #691 (ac41122) into main (4dccbc0) will decrease coverage by 0.00%. The diff coverage is 75.00%.

@@            Coverage Diff             @@
##             main     #691      +/-   ##
==========================================
- Coverage   84.93%   84.92%   -0.01%     
==========================================
  Files         157      157              
  Lines        9098     9101       +3     
==========================================
+ Hits         7727     7729       +2     
- Misses       1371     1372       +1     
Impacted Files Coverage Ξ”
...ces/XcodeProj/Objects/Targets/PBXProductType.swift 54.28% <50.00%> (-1.60%) :arrow_down:
...rojTests/Objects/Targets/PBXProductTypeTests.swift 100.00% <100.00%> (ΓΈ)

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 4dccbc0...ac41122. Read the comment docs.

mtj0928 commented 2 years ago

Thank you for your discussion, I renamed the new case and pushed the changesπŸ˜„

freddi-kit commented 2 years ago

Hello, @kwridan, when is it released as new version?

kwridan commented 2 years ago

Let's wait till #694 is merged then we can do a release with a few of the Xcode 14 related items within it.