[GitHub] [cordova-ios] brodybits commented on pull request #795: [minor] fix *-Info.plist & config.xml functionality

[GitBox <gi...@apache.org>]

brodybits commented on pull request #795:
URL: https://github.com/apache/cordova-ios/pull/795#issuecomment-643408727


   The good news is that I would no longer consider this to be a breaking change. If I would remove the xcworkspace file and then build from the command line, it results in an error from xcodebuild. (I was able to remove the xcworkspace file and then build & run from Xcode, which I think is a side point.)
   
   I do think this is too much change for a patch, though, should probably target a minor release.
   
   I did try merging. The changes did not seem to work after merging, and some of the tests need rework due to removing shelljs. I think we should really rebase and rework. I will try to fit this in as soon as I can.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org