Fixes #170

* Revert "Use multirun (#160)"

This reverts commit 6df95c60bac5057e0ef0fc2eabcf33e46b5b3e06.
...

* refactor: prefix public API symbols with lint_

* chore: docgen

The latter is a global namespace, and also leaked into a test where it shouldn't have been needed

* refactor: use rules_multirun rather than implement our own

* Update .gitignore

* fix: sy...

* refactor: use rules_multitool to give defaults for formatter tools

So far this just replace...

Co-authored-by: alexeagle <[email protected]>

Co-authored-by: alexeagle <[email protected]>

* refactor: break up tools/ into tools/format and tools/lint

Suggested during API review, as ...

Thanks @jdkato for the fix in https://github.com/errata-ai/vale/commit/2139c4176a4d2e62d7dfb95dc...

* feat: add yamlfmt for YAML

Fixes #152

* chore: green up

Co-authored-by: alexeagle <[email protected]>

Fixes #80

step on the road to parallelizing the formatters

This is what the example setup uses, and it's what I had to use to
get this to work in our work...

* chore: add vale binary

* chore: start the vale aspect

* chore: workaround how vale locat...

The 'stylish' report is kinda hard to maintain a fork of it.
I broke the errorformat parser by ...

* fix(HCL): back out from formatting all HCL files

The tool we are using (terraform fmt) does...

clang-format violations are treated as warnings, so the exit code is 0
unless Werror is specified.

Co-authored-by: alexeagle <[email protected]>

* fix undocumented dependency on aspect_bazel_lib for WORKSPACE users

I noticed that the inst...

We don't have any Go sources in rules_lint so there's no reason we should install a Go toolchain...

Merge https://github.com/bazel-contrib/rules-template/pull/105

Hint that only one formatter can be chosen per language

gofumpt is a popular alternative formatter for Go that is compatible
with gofmt but stricter.

git ls-files seems to match against filename strings exactly, and
doesn't use the same logic as ...

* ci: add test coverage for format binary

This is a beginning, if the pattern works I'll appl...

Without this, errors don't end up in the report file and are thus only
displayed on the first run.

These "example commits" aren't up-to-date with what's in the repo, and it's not feasible to keep...

List supported tools earlier

Co-authored-by: alexeagle <[email protected]>

* refactor: overhaul of format.sh

Use GitHub Linguist to define the file extensions for each ...

* chore: upgrade to Bazel 7

* chore: choose earlier llvm

The latest doesn't appear to be s...

* feat: Add clang-format for formatting C/C++

* Update docs

* Add llvm_toolchain to WORKSP...

Co-authored-by: alexeagle <[email protected]>

* Add golangci-lint as an aspect

* Update .golangci.yaml

Paste documentation about the "do...

* Enable shellcheck aspect for sh_binary

* Update shellcheck.bzl

* Update shellcheck.bzl

* fix(formatter): markdown needs a distinct tool entry

Otherwise it overloads the one for Jav...

Fixes #90

This release stops using symbols that were removed in bazel-lib 2.0

Co-authored-by: alexeagle <[email protected]>

Test files aren't being linted with the previous approach.

* Patch stuff

* chore: reduce delta

* refactor: separate action factory for --fix

* tem...

It's pretty confusing that it's lumped under 'JavaScript' currently.

Fixes #81

* Exclude deleted files from format

Fixes https://github.com/aspect-build/rules_lint/issues/8...

Co-authored-by: alexeagle <[email protected]>

* fix: main is red because ruff was mirrored incorrectly

* fix: account for ruff assets namin...

Co-authored-by: alexeagle <[email protected]>

* docs: explain which production of ts_project to lint

Fixes #74

* fix: typings not typech...

I don't have time to change rules_swift and plan to close this issue upstream.

* feat: allow multiple eslint configs in one aspect definition

Equivalent of #60 which did th...

I saw us making this change in other repos so it should be consistent.
Maybe it also explains w...

* refactor: reduce duplication in formatter lang:tool mapping

* chore: docgen

* fix: Respect exclude when formatting with ruff

As per discussion here: https://github.com/a...

* fix: don't error for js_binary w no srcs

* Update eslint.bzl

Co-authored-by: alexeagle <[email protected]>

Fixes #42

Fixes #17

We're using this as part of Aspect Workflows so we plan to stick with it.

Previously we always directed the tools output into a report file, but when you ask for the exit...

* fix: only present reports from the current bazel run

Thanks to @mortenmj for providing the ...

* feat: allow multiple ruff config files

As documented at https://docs.astral.sh/ruff/configu...

* style: recommend format as an alias in root BUILD

* docs: code review comment

* docs: co...

Co-authored-by: alexeagle <[email protected]>

It knows to create a commit first, which the gh CLI doesn't

Similar to https://github.com/aspect-build/bazel-super-formatter/pull/47/commits/a416c6b3744ce9f...

* feat: auto-mirror ruff releases

If this pattern works well, I'll roll out to all our rulese...

We can't exclude examples because that's the bcr test module.

These commonly contain an entry point or "main" which should be linted,
but not contained in an...

As fix-mode does not return a non-zero status, it will not block submission when it detects form...

`aspect/5.7.2` didn't include the `lint` command and `bazel lint src:all` fails with

```
Err...

If the linter binaries aren't visible to targets under `src/`, linting fails with

```
❯ baze...

The `example/` repo fails to lint because the `ruff` binary it provides is a source file.

``...

* Replace prettier-plugin-sh with shfmt

* Update BUILD

---------

Co-authored-by: Alex E...

Allows running multiple linters on the same target.