Cat Chen

Source in Markdown?

1

Is it possible to have source in Markdown? It would be really nice to have better format and getting plain text back from Markdown is easy.

Example: https://github.com/CatChen/eslint-suggestion-action/actions/runs/4968279456/jobs/8890782775?pr=110 This happens when `targets` input is expanded to too many files in the command line.

The action just fails in execution when there's no ESLint.

When deleting the last line that uses a variable, it introduced an ESLint error to the unused `const`, `import` or whatnot. Because that's out of the diff hunk it's not...

The existing code doesn’t support pagination and will have problem handling a PR with too many files. https://github.com/CatChen/eslint-suggestion-action/blob/4c6c9e8f1334d209364d584e947b82eaece3ab43/src/index.ts#L196-L200 We can do something like this: https://github.com/actions/labeler/blob/ed55e8ab1fa323b4a53afb55132c065aa7a9f8f4/src/labeler.ts#L78-L84 The pagination API can be...

http://wiki.commonjs.org/wiki/Promises

`module.exports` in every module will overwrite each other in simple code concatenation, making these built file useless: - https://github.com/CatChen/jsHelpers/blob/master/build/jshelpers.js - https://github.com/CatChen/jsHelpers/blob/master/build/jshelpers-min.js

Better error message when the app isn't installed to the repo

1

The error message `Error: Cannot read properties of undefined (reading 'id')` isn't very descriptive. I looked into this action's source code and realized that the app wasn't properly installed to...