apache / cordova-ios

Apache Cordova iOS
https://cordova.apache.org/
Apache License 2.0
2.15k stars 987 forks source link

chore: remove long-unused codeSignResourceRules option #1308

Closed dpogue closed 1 year ago

dpogue commented 1 year ago

Platforms affected

iOS

Motivation and Context

Xcode hasn't supported specifying codesigning resource rules since pre-Mavericks, so this serves no purpose anymore. Closes #563.

Description

Remove support for the --codeSignResourceRules option.

Testing

All unit tests pass.

Checklist

codecov-commenter commented 1 year ago

Codecov Report

Merging #1308 (6f41fbb) into master (3d6c71a) will increase coverage by 0.03%. The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1308      +/-   ##
==========================================
+ Coverage   78.48%   78.51%   +0.03%     
==========================================
  Files          15       15              
  Lines        1780     1778       -2     
==========================================
- Hits         1397     1396       -1     
+ Misses        383      382       -1     
Impacted Files Coverage Δ
lib/build.js 67.21% <ø> (+0.18%) :arrow_up:

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