Skip to content

Code coverage report fixes, Action updates, added Health Score check … #4

Code coverage report fixes, Action updates, added Health Score check …

Code coverage report fixes, Action updates, added Health Score check … #4

GitHub Actions / Health Score completed May 13, 2024 in 0s

-1000

-1000 health score points

Details

Score Breakdown

Problematic Comments

Each problematic comment (i.e. comments with TODO, HACK or FIXME in it) contributes -100 points to the health score.

  • // TODO: type this to account for variable return options?
  • // TODO: should we check that this is a ValidEventType at runtime?
  • // TODO: should we throw if this cannot be found? Alternatively / in addition, if calls below to complete* APIs fail, maybe we should throw then?
  • // TODO: in the future, if we add more of this kind of logging, then perhaps worth considering moving to a structured logger with different log levels
  • // TODO: latest version of std does not have a io/utils.ts
  • // TODO: Update to import from deno-slack-hooks instead
  • // TODO: does the following need to be wrapped in a try/catch too?
  • // TODO: I think we want to be intentional about how we use try/catch here.
  • // TODO: where would this log show up? Since this runs on Slack hosting in a lambda context
  • // TODO: if userland code has console.logs _outside_ of a function handler, at the top level scope of the module, _and_ the current execution is a local run and not a deployed-on-ROSI run,