Test json validate

andrew-codechimp · Dec 12, 2023 · 9884293 · 9884293
1 parent 67e15e2
commit 9884293
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 4 deletions.
diff --git a/.github/workflows/json_validate.yml b/.github/workflows/json_validate.yml
@@ -1,6 +1,7 @@
 name: Validate JSONs
 
 on:
+  workflow_dispatch:
   push:
     paths:
     - 'custom_components/battery_notes/data/**'

diff --git a/custom_components/battery_notes/data/library.json b/custom_components/battery_notes/data/library.json
@@ -1,4 +1,5 @@
 {
+    "$schema": "../../../schema.json",
     "version": 1,
     "devices": [
         {

diff --git a/schema.json b/schema.json
@@ -5,6 +5,10 @@
     "description": "Library of battery types for devices",
     "type": "object",
     "properties": {
+        "$schema": {
+            "description": "Schema reference",
+            "type": "string"
+        },
         "version": {
             "description": "Version of the library schema",
             "type": "integer"
@@ -21,23 +25,29 @@
                 ],
                 "properties": {
                     "manufacturer": {
-                        "type": "string"
+                        "type": "string",
+                        "description": "The manufacturer of the device as it appears in Home Assistant."
                     },
                     "model": {
-                        "type": "string"
+                        "type": "string",
+                        "description": "The model of the device as it appears in Home Assistant."
                     },
                     "battery_type": {
-                        "type": "string"
+                        "type": "string",
+                        "description": "The type of battery for the device. Should be the most common naming for general batteries, and the IEC naming for battery cells according to Wikipedia."
                     },
                     "battery_quantity": {
-                        "type": "integer"
+                        "type": "integer",
+                        "exclusiveMinimum": 1,
+                        "description": "The number of batteries required by the device. If a device only has one battery this property should be omitted."
                     }
                 }
             }
         }
     },
     "additionalProperties": false,
     "required": [
+        "$schema",
         "version",
         "devices"
     ]