Add ubuntu and windows CI actions (#8)

* Add ubuntu and windows CI actions

* Update file spacing

* Update windows.yml and fix multiple platform cache

* Use correct boolean operator

* Limit PersitableCache to apple platforms

* Limit ExampleTests

* Limit PersistableCacheTest

* Update code for Windows support

* Make Windows PropertyWrapper unique inits

* Limit some tests in ExpiringCacheTest from windows

Author: 0xLeif

.github/workflows/CI.yml renamed to .github/workflows/macOS.yml

@@ -1,4 +1,4 @@
-name: CI
+name: macOS
 
 on:
   push:
@@ -8,7 +8,7 @@ on:
 
 jobs:
   build:
-    runs-on: macos-12
+    runs-on: macos-latest
 
     steps:
     - uses: actions/checkout@v3

.github/workflows/ubuntu.yml

@@ -0,0 +1,18 @@
+name: Ubuntu
+
+on:
+  push:
+    branches: [ "main" ]
+  pull_request:
+    branches: [ "main" ]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v3
+    - name: Build
+      run: swift build -v
+    - name: Run tests
+      run: swift test -v

.github/workflows/windows.yml

@@ -0,0 +1,20 @@
+name: Windows
+
+on:
+  push:
+    branches: [ "main" ]
+  pull_request:
+    branches: [ "main" ]
+
+jobs:
+  build:
+    runs-on: windows-latest
+    steps:
+      - uses: compnerd/gha-setup-swift@main
+        with:
+          branch: swift-5.6-release
+          tag: 5.6-RELEASE
+  
+      - uses: actions/checkout@v2
+      - run: swift build
+      - run: swift test

Sources/Cache/Cache/Cache+subscript.swift

@@ -12,7 +12,7 @@ extension Cache {
             get(key, as: Value.self)
         }
         set(newValue) {
-            guard let newValue else {
+            guard let newValue = newValue else {
                 return remove(key)
             }
 

Sources/Cache/Cache/Cache.swift

@@ -13,13 +13,17 @@ import Foundation
  let age = cache.get("age") // age is now 100
  ```
  */
-open class Cache<Key: Hashable, Value>: Cacheable, ObservableObject {
+open class Cache<Key: Hashable, Value>: Cacheable {
 
     /// Lock to synchronize the access to the cache dictionary.
     fileprivate var lock: NSLock
 
+    #if os(Linux) || os(Windows)
+    fileprivate var cache: [Key: Value] = [:]
+    #else
     /// The actual cache dictionary of key-value pairs.
     @Published fileprivate var cache: [Key: Value] = [:]
+    #endif
 
     /**
      Initializes a new `Cache` instance with an optional dictionary of initial key-value pairs.
@@ -154,3 +158,7 @@ open class Cache<Key: Hashable, Value>: Cacheable, ObservableObject {
         return cache.values(ofType: Output.self)
     }
 }
+
+#if !os(Linux) && !os(Windows)
+extension Cache: ObservableObject { }
+#endif

Sources/Cache/Cache/ExpiringCache+subscript.swift

@@ -12,7 +12,7 @@ extension ExpiringCache {
             get(key, as: Value.self)
         }
         set(newValue) {
-            guard let newValue else {
+            guard let newValue = newValue else {
                 return remove(key)
             }
 

Sources/Cache/Cache/PersistableCache.swift

@@ -1,3 +1,4 @@
+#if !os(Linux) && !os(Windows)
 import Foundation
 
 /**
@@ -142,3 +143,4 @@ private extension URL {
         return appending(path: name)
     }
 }
+#endif

Sources/Cache/JSON/JSON+subscript.swift

@@ -12,7 +12,7 @@ extension JSON {
             get(key, as: Any.self)
         }
         set(newValue) {
-            guard let newValue else {
+            guard let newValue = newValue else {
                 return remove(key)
             }
 

Sources/Cache/PropertyWrappers/Cached.swift

@@ -42,6 +42,8 @@
         }
     }
 
+
+    #if !os(Windows)
     /**
     Initializes a new instance of the `Cached` property wrapper.
 
@@ -59,4 +61,23 @@
         self.cache = cache
         self.defaultValue = defaultValue
     }
+    #else
+    /**
+    Initializes a new instance of the `Cached` property wrapper.
+
+    - Parameters:
+       - key: The key associated with the value in the cache.
+       - cache: The cache instance to retrieve the value from.
+       - defaultValue: The default value to be used if the value is not present in the cache.
+    */
+    public init(
+        key: Key,
+        using cache: Cache<Key, Any>,
+        defaultValue: Value
+    ) {
+        self.key = key
+        self.cache = cache
+        self.defaultValue = defaultValue
+    }
+    #endif
 }

Sources/Cache/PropertyWrappers/OptionallyCached.swift

@@ -38,17 +38,18 @@
             cache.get(key, as: Value.self)
         }
         set {
-            guard let newValue else {
+            guard let newValue = newValue else {
                 return cache.remove(key)
             }
 
             cache.set(value: newValue, forKey: key)
         }
     }
 
+    #if !os(Windows)
     /**
      Initializes a new instance of the `OptionallyCached` property wrapper.
-     
+
      - Parameters:
      - key: The key associated with the value in the cache.
      - cache: The cache instance to retrieve the value from. The default is `Global.cache`.
@@ -60,4 +61,20 @@
         self.key = key
         self.cache = cache
     }
+    #else
+    /**
+     Initializes a new instance of the `OptionallyCached` property wrapper.
+
+     - Parameters:
+     - key: The key associated with the value in the cache.
+     - cache: The cache instance to retrieve the value from.
+     */
+    public init(
+        key: Key,
+        using cache: Cache<Key, Any>
+    ) {
+        self.key = key
+        self.cache = cache
+    }
+    #endif
 }