[Feature Request] Add ability to specify own error handler about croner HOT 6 CLOSED

Will have a look at that 👍

from croner.

Thanks, I didn't expect to see it released that quickly!
I'm using TypeScript and I noticed a small typing issue:

cron("* * * * * *", () => {
    throw new Error("err!");
  },
  {
    // TS7006: Parameter 'e' implicitly has an 'any' type. (appears if strict mode is enabled)
    // Can be fixed by adding `e: any`
    catch(e) {
      console.error("Error!", e);
    },
  }
);

It'd be beneficial for TS users to define the arguments explicitly, so the compiler knows there's an error parameter:
catch?: boolean | ((error: unknown) => void);

from croner.

(Hopefully) resolved in 5.4.1 which will be released any minute +1

Yep, I can confirm it's fixed. Thanks!

from croner.

Released in 5.4.0 - give it a go!

Example (running on Deno):

import Cron from "https://deno.land/x/[email protected]/dist/croner.min.mjs";

const onError = (e) => console.error('Oh shit - ', e);

Cron("* * * * * *", { catch: onError },
	() => {
		console.log("This will run every second.");
		throw new Error("t");
	}
);

from croner.

Thanks for reporting (again)! Will have a look at that for 5.4.1.

from croner.

(Hopefully) resolved in 5.4.1 which will be released any minute 👍

from croner.