Api

[veeti-k]

No description provided.

[AriPerkkio]

En ehtinyt vielä ihan läpi koluta, mutta alla parit kommentit.

[AriPerkkio]

Parit kommentit alla. En vieläkään ihan loppuun asti ehtinyt katsoa.

[AriPerkkio]

Parissa virheenkäsittelyssä turhaa toistoa: parsitaan virheet, päivitetään actionin status, päivitetään issuen kommentti ja ajetaan return. Sen sijaan voisi wrapata try-catch:llä koko blockin, ja heittää virheet kun niitä tulee. Lopussa catchillä sitten tehdä nuo käsittelyt.

Tai jos ei try-catch innosta niin jotain:

void main().catch(error => {
  // Update action status
  // Update issue comment
  // exit
});

[AriPerkkio]

async function run() {
  if(validationError) {
    throw new Error('Validation failed');
  }

  if(anotherError) {
    throw new Error('Another error');
  }
}

async function main() {
  try {
    await run();
  } catch (error) {
    octokit.updateIssueComment(...)
    core.setError(...)
  }
}