tuist / XcodeProj

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

Add shellToInvoke to XCScheme.ExecutionAction #721

Closed CrazyFanFan closed 2 years ago

CrazyFanFan commented 2 years ago

Short description πŸ“

If a post_actions specifies shellToInvoke, this field will be lost after rewriting with XCScheme;

Solution πŸ“¦

Add property shellToInvoke to XCScheme.ExecutionAction to fix shellToInvoke missing after editing XCProject.

kwridan commented 2 years ago

@all-contributors add @CrazyFanFan for code

allcontributors[bot] commented 2 years ago

@kwridan

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

codecov-commenter commented 2 years ago

Codecov Report

Merging #721 (9b77208) into main (c36d3a8) will increase coverage by 0.00%. The diff coverage is 84.61%.

@@           Coverage Diff           @@
##             main     #721   +/-   ##
=======================================
  Coverage   84.90%   84.90%           
=======================================
  Files         158      158           
  Lines        9241     9249    +8     
=======================================
+ Hits         7846     7853    +7     
- Misses       1395     1396    +1     
Impacted Files Coverage Ξ”
...es/XcodeProj/Scheme/XCScheme+ExecutionAction.swift 72.72% <81.81%> (+2.35%) :arrow_up:
Tests/XcodeProjTests/Scheme/XCSchemeTests.swift 100.00% <100.00%> (ΓΈ)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more