Skip to content

test: ignore pr#207

Closed
dcreey wants to merge 5 commits intomainfrom
dcreey/test-lint
Closed

test: ignore pr#207
dcreey wants to merge 5 commits intomainfrom
dcreey/test-lint

Conversation

@dcreey
Copy link
Copy Markdown
Contributor

@dcreey dcreey commented May 13, 2025

No description provided.

Comment thread .github/workflows/ci.yml Fixed
@dcreey dcreey closed this May 13, 2025
@dcreey dcreey reopened this May 13, 2025
mattrandallbecker pushed a commit to abcxyz/actions that referenced this pull request May 14, 2025
Add a new action for linting javascript/typescript/node based code.

* Runs on every package.json in the repo. Shards to one vm per
package.json
* Runs `npm run lint` if a lint command is defined in the package.json
* Otherwise runs `npx eslint .` if an eslint config is present.
* Otherwise does nothing and instead prints a notice to add a "lint"
command to your package.json or add an eslint config.
* NOTE: I considered using [gts](https://github.com/google/gts) - but it
just adds a `lint` command to your package.json and is not CI friendly
as it generates a lot of source code if you haven't used it before in a
repo.
* TODO: lint standalone js

Tested on github-token-minter by manually adding:
abcxyz/github-token-minter#207
@dcreey dcreey changed the title test: auto lint js test: ignore pr May 16, 2025
@dcreey dcreey marked this pull request as ready for review May 16, 2025 15:05
@dcreey dcreey requested a review from a team as a code owner May 16, 2025 15:05
@dcreey dcreey requested review from pjh and verbanicm and removed request for pjh and verbanicm May 16, 2025 15:05
@dcreey dcreey closed this Jun 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants