Configuration IO is synchronous

[RA80533]

• codeql-action/src/config-utils.ts

  Line 1074 in f00bafb

  async function saveConfig(config: Config, logger: Logger) {

• codeql-action/src/config-utils.ts

  Line 1087 in f00bafb

  export async function getConfig(

Both are annotated as asynchronous functions but neither utilizes asynchronous IO.

[robertbrignull]

Nice catch. Not sure exactly what happened here but I'd guess they were genuinely asynchronous at some point in the past. This looks like a good candidate for a CodeQL query to find unnecessary async methods. I'm surprised that doesn't exist already to be honest as it sounds like it could be broadly useful.