From 528fcb3b1052bfa0915df65b620823e9cf3d5ebf Mon Sep 17 00:00:00 2001
From: Vinit Kumar <vinit.kumar@kidskonnect.nl>
Date: Wed, 20 Apr 2022 02:10:35 +0530
Subject: [PATCH 1/2] fix: issue with wrong output for boolean list

---
 examples/booleanjson2.json | 5 +++++
 json2xml/dicttoxml.py      | 9 ++++++---
 tests/test_json2xml.py     | 9 +++++++++
 3 files changed, 20 insertions(+), 3 deletions(-)
 create mode 100644 examples/booleanjson2.json

diff --git a/examples/booleanjson2.json b/examples/booleanjson2.json
new file mode 100644
index 00000000..95a35a8e
--- /dev/null
+++ b/examples/booleanjson2.json
@@ -0,0 +1,5 @@
+{
+      "boolean_list": [true, false],
+      "number_array": [1, 2, 3],
+      "string_array": ["a", "b", "c"]
+}
\ No newline at end of file
diff --git a/json2xml/dicttoxml.py b/json2xml/dicttoxml.py
index 489d55ff..6c3b0e73 100755
--- a/json2xml/dicttoxml.py
+++ b/json2xml/dicttoxml.py
@@ -137,6 +137,7 @@ def convert(obj, ids, attr_type, item_func, cdata, item_wrap, parent="root"):
 
     LOG.info(f'Inside convert(). obj type is: "{type(obj).__name__}", obj="{str(obj)}"')
 
+
     item_name = item_func(parent)
 
     # since bool is also a subtype of number.Number and int, the check for bool
@@ -265,7 +266,11 @@ def convert_list(items, ids, parent, attr_type, item_func, cdata, item_wrap):
             f'Looping inside convert_list(): item="{str(item)}", item_name="{item_name}", type="{type(item).__name__}"'
         )
         attr = {} if not ids else {"id": f"{this_id}_{i + 1}"}
-        if isinstance(item, (numbers.Number, str)):
+
+        if isinstance(item, bool):
+            addline(convert_bool(item_name, item, attr_type, attr, cdata))
+
+        elif isinstance(item, (numbers.Number, str)):
             if item_wrap:
                 addline(
                     convert_kv(
@@ -298,8 +303,6 @@ def convert_list(items, ids, parent, attr_type, item_func, cdata, item_wrap):
                 )
             )
 
-        elif isinstance(item, bool):
-            addline(convert_bool(item_name, item, attr_type, attr, cdata))
 
         elif isinstance(item, dict):
             item_dict_str = convert_dict(
diff --git a/tests/test_json2xml.py b/tests/test_json2xml.py
index 872ee323..54ef4d50 100644
--- a/tests/test_json2xml.py
+++ b/tests/test_json2xml.py
@@ -184,3 +184,12 @@ def test_read_boolean_data_from_json(self):
         dict_from_xml = xmltodict.parse(result)
         assert dict_from_xml["all"]["boolean"]["#text"] != 'True'
         assert dict_from_xml["all"]["boolean"]["#text"] == 'true'
+
+
+    def test_read_boolean_data_from_json2(self):
+        """Test correct return for boolean types."""
+        data = readfromjson("examples/booleanjson2.json")
+        result = json2xml.Json2xml(data).to_xml()
+        dict_from_xml = xmltodict.parse(result)
+        assert dict_from_xml["all"]["item"][0]["#text"] != 'True'
+        assert dict_from_xml["all"]["item"][0]["#text"] == 'true'

From 55403507ab845e514dc8223c4408ab9e66746d02 Mon Sep 17 00:00:00 2001
From: Vinit Kumar <vinit.kumar@kidskonnect.nl>
Date: Wed, 20 Apr 2022 02:20:09 +0530
Subject: [PATCH 2/2] feat: bump version and generate changelog

---
 HISTORY.rst          | 6 ++++++
 json2xml/__init__.py | 2 +-
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/HISTORY.rst b/HISTORY.rst
index 729feb10..72f56259 100644
--- a/HISTORY.rst
+++ b/HISTORY.rst
@@ -1,6 +1,12 @@
 History
 =======
 
+3.17.1 / 2022-04-20
+===================
+
+  * fix: issue with wrong output for boolean list
+  * fix: pull requests also trigger action runs
+
 3.17.0 / 2022-04-18
 ===================
 
diff --git a/json2xml/__init__.py b/json2xml/__init__.py
index 68d1b738..1f270518 100644
--- a/json2xml/__init__.py
+++ b/json2xml/__init__.py
@@ -2,4 +2,4 @@
 
 __author__ = """Vinit Kumar"""
 __email__ = "mail@vinitkumar.me"
-__version__ = "3.17.0"
+__version__ = "3.17.1"