dpogue commented on code in PR #1546:
URL: https://github.com/apache/cordova-ios/pull/1546#discussion_r2157821797
##########
CordovaLib/CordovaLib.docc/upgrading-8.md:
##########
@@ -152,6 +152,28 @@ if webView.responds(to: scrollViewSelector) {
This updated code is compatible with existing versions of Cordova iOS.
+# Precompiled Prefix Header deprecation
+
+Previously, Cordova projects included a precompiled prefix header that
automatically imported the `Foundation` and `UIKit` frameworks. This made these
frameworks available globally, without requiring explicit imports in each
Objective-C file.
+
+While this may have offered convenience, it also introduced an implicit
dependency on the Cordova-managed prefix header and prefix headers have
gradually been replaced with module imports in Objective-C and were never
supported in Swift.
+
+To align with Xcode defaults and improve long-term maintainability, the
precompiled prefix header has been removed from generated Cordova app projects.
While this may be a breaking change for some plugins, developers are now
expected to explicitly declare the frameworks their code depends on by adding
the appropriate import statements directly in their source files.
+
+**Objective-C Example:**
+
+```objc
Review Comment:
```suggestion
```objc
// New code (Objective-C)
```
##########
CordovaLib/CordovaLib.docc/upgrading-8.md:
##########
@@ -152,6 +152,28 @@ if webView.responds(to: scrollViewSelector) {
This updated code is compatible with existing versions of Cordova iOS.
+# Precompiled Prefix Header deprecation
+
+Previously, Cordova projects included a precompiled prefix header that
automatically imported the `Foundation` and `UIKit` frameworks. This made these
frameworks available globally, without requiring explicit imports in each
Objective-C file.
+
+While this may have offered convenience, it also introduced an implicit
dependency on the Cordova-managed prefix header and prefix headers have
gradually been replaced with module imports in Objective-C and were never
supported in Swift.
+
+To align with Xcode defaults and improve long-term maintainability, the
precompiled prefix header has been removed from generated Cordova app projects.
While this may be a breaking change for some plugins, developers are now
expected to explicitly declare the frameworks their code depends on by adding
the appropriate import statements directly in their source files.
+
+**Objective-C Example:**
+
+```objc
+#import <Foundation/Foundation.h>
+#import <UIKit/UIKit.h>
+```
+
+**Swift Example:**
+
+```swift
Review Comment:
```suggestion
```swift
// New code (Swift)
```
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
