fix: PR comments

Author: ahmedAlaaInstabug

src/modules/APM.ts

@@ -62,8 +62,7 @@ export const startExecutionTrace = async (name: string): Promise<Trace> => {
   const id = await NativeAPM.startExecutionTrace(name, timestamp);
 
   if (!id) {
-    console.error(TRACE_NOT_STARTED_APM_NOT_ENABLED);
-    return Promise.reject(TRACE_NOT_STARTED_APM_NOT_ENABLED);
+    throw new Error(TRACE_NOT_STARTED_APM_NOT_ENABLED);
   }
 
   return new Trace(id, name);

src/modules/Instabug.ts

@@ -381,7 +381,7 @@ export const setReproStepsConfig = (config: ReproConfig) => {
  */
 export const setUserAttribute = (key: string, value: string) => {
   if (!key || typeof key !== 'string' || typeof value !== 'string') {
-    console.error('Invalid param, Expected String');
+    console.error('IBG-RN: Expected key and value passed to setUserAttribute to be of type string');
     return;
   }
   NativeInstabug.setUserAttribute(key, value);
@@ -405,7 +405,8 @@ export const getUserAttribute = async (key: string): Promise<string | null> => {
  */
 export const removeUserAttribute = (key: string) => {
   if (!key || typeof key !== 'string') {
-    console.error('Invalid param, Expected String');
+    console.error('IBG-RN: Expected the key passed to removeUserAttribute to be of type string');
+
     return;
   }
   NativeInstabug.removeUserAttribute(key);