Set product to .dynamic when bridging

Author: marcprux

Package.swift

@@ -12,7 +12,7 @@ let package = Package(
     defaultLocalization: "en",
     platforms: [.iOS(.v16), .macOS(.v13), .tvOS(.v16), .watchOS(.v9), .macCatalyst(.v16)],
     products: [
-        .library(name: "SkipKeychain", type: .dynamic, targets: ["SkipKeychain"]),
+        .library(name: "SkipKeychain", targets: ["SkipKeychain"]),
     ],
     dependencies: [
         .package(url: "https://source.skip.tools/skip.git", from: "1.2.1"),
@@ -29,4 +29,9 @@ if ProcessInfo.processInfo.environment["SKIP_BRIDGE"] ?? "0" != "0" {
     package.targets.forEach({ target in
         target.dependencies += [.product(name: "SkipBridge", package: "skip-bridge")]
     })
+    // all library types must be dynamic to support bridging
+    package.products = package.products.map({ product in
+        guard let libraryProduct = product as? Product.Library else { return product }
+        return .library(name: libraryProduct.name, type: .dynamic, targets: libraryProduct.targets)
+    })
 }

README.md

@@ -61,6 +61,19 @@ Kotlin JUnit tests in the Robolectric Android simulation environment.
 Parity testing can be performed with `skip test`,
 which will output a table of the test results for both platforms.
 
+## Contributing
+
+We welcome contributions to this package in the form of enhancements and bug fixes.
+
+The general flow for contributing to this and any other Skip package is:
+
+1. Fork this repository and enable actions from the "Actions" tab
+2. Check out your fork locally
+3. When developing alongside a Skip app, add the package to a [shared workspace](https://skip.tools/docs/contributing) to see your changes incorporated in the app
+4. Push your changes to your fork and ensure the CI checks all pass in the Actions tab
+5. Add your name to the Skip [Contributor Agreement](https://github.com/skiptools/clabot-config)
+6. Open a Pull Request from your fork with a description of your changes
+
 ## License
 
 This software is licensed under the