Apply review suggestions

Author: christiand93

docs/general-usage/04-custom-fields.md

@@ -41,11 +41,11 @@ info("Test data", {"field": 42});
 
 ### SAP Cloud Logging
 
-When using the library with SAP Cloud Logging custom fields get added as top-level fields.
+When using the library with SAP Cloud Logging, custom fields get added as top-level fields.
 Registering custom fields is not required in this case.
 
 By default all custom field values get stringified, which can help to prevent field type mismatches when indexing to SAP Cloud Logging.
-However, if you want to retain the original type of your custom fields you can change the type conversion configuration accordingly:
+However, if you want to retain the original type of your custom fields, you can change the type conversion configuration accordingly:
 
 ```js
 log.setCustomFieldsTypeConversion(CustomFieldsTypeConversion.Retain)

src/lib/logger/recordFactory.ts

@@ -115,10 +115,10 @@ export default class RecordFactory {
                 || record.payload[key] != null || this.config.isSettable(key)) {
                 // Stringify, if conversion type 'stringify' is selected and value is not a string already.
                 if (customFieldsTypeConversion == CustomFieldsTypeConversion.Stringify && (typeof value) != "string") {
-                    record.payload[key] = jsonStringifySafe(value);;
+                    record.payload[key] = jsonStringifySafe(value);
                 } else {
                     record.payload[key] = value;
-                }               
+                }
             }
 
             if ([CustomFieldsFormat.ApplicationLogging, CustomFieldsFormat.All].includes(customFieldsFormat)) {
@@ -131,7 +131,7 @@ export default class RecordFactory {
             }
         });
 
-        //writes custom fields in the correct order and correlates i to the place in registeredCustomFields
+        // Write custom fields in the correct order and correlates i to the place in registeredCustomFields
         if (Object.keys(indexedCustomFields).length > 0) {
             let res: any = {};
             res.string = [];

src/test/acceptance-test/custom-fields.test.js

@@ -145,7 +145,7 @@ describe('Test custom fields', function () {
                 log.logMessage("info", "test-message");
             });
 
-            it('logs custom fields as strings', function () {
+            it('logs custom fields with their retained type', function () {
                 expect(lastOutput).to.have.property('field-a', 42).that.is.a('number');
                 expect(lastOutput).to.have.property('field-b', true).that.is.a('boolean');
                 expect(lastOutput).to.have.property('field-c').that.deep.equals({"key":"value"})