Skip to content

Tags: aspect-build/rules_lint

Tags

v1.13.0

Toggle v1.13.0's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
fix: add new bzl_library for 'ty' with dependencies (#675)

otherwise loading it is an error for stardoc

v1.12.0

Toggle v1.12.0's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
feat: Add cppcheck lint aspect (#617)

Add `cppcheck` as `lint` aspect as per
#569.
A local test can be ran using: `cd example; bazel lint //src:hello_cc`
The aspect supports human readable output and `xml` output, e.g.
`cd example; bazel build //...
--aspects=//tools/lint:linters.bzl%cppcheck
--output_groups=rules_lint_xml`

### Changes are visible to end-users: yes

- Searched for relevant documentation and updated as needed: yes
- Breaking change (forces users to change their own code or config): no
- Suggested release notes appear below: yes

"Add cppcheck linter"

### Test plan

- Manual testing: `cd example; bazel build //...
--aspects=//tools/lint:linters.bzl%cppcheck
--output_groups=rules_lint_xml`: This will produce following output:
```
INFO: Analyzed 194 targets (0 packages loaded, 0 targets configured).
INFO: Found 194 targets...
Aspect //tools/lint:linters.bzl%cppcheck of //src:hello_cc up-to-date:
  bazel-bin/src/hello_cc.AspectRulesLintCppCheck.xml
Aspect //tools/lint:linters.bzl%cppcheck of //src/cpp/main:hello-greet up-to-date:
  bazel-bin/src/cpp/main/hello-greet.AspectRulesLintCppCheck.xml
Aspect //tools/lint:linters.bzl%cppcheck of //src:hello_c up-to-date:
  bazel-bin/src/hello_c.AspectRulesLintCppCheck.xml
Aspect //tools/lint:linters.bzl%cppcheck of //src/cpp/lib:hello-time up-to-date:
  bazel-bin/src/cpp/lib/hello-time.AspectRulesLintCppCheck.xml
Aspect //tools/lint:linters.bzl%cppcheck of //src/cpp/main:hello-world up-to-date:
  bazel-bin/src/cpp/main/hello-world.AspectRulesLintCppCheck.xml
INFO: Elapsed time: 0.227s, Critical Path: 0.01s
INFO: 1 process: 1 internal.
INFO: Build completed successfully, 1 total action
```

---------

Co-authored-by: Alex Eagle <alex@aspect.build>

v1.11.0

Toggle v1.11.0's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Update formatting.md

v1.10.2

Toggle v1.10.2's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
fix: need newer publish-to-bcr

The 1.10.1 release has missing interpolation in docs_url

v1.10.1

Toggle v1.10.1's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
fix: add docs_url

v1.10.0

Toggle v1.10.0's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
fix: Shell formatter finds files without extension (#622)

This lets `shfmt` format shell scripts identified only by their shebang
but without any extension.

Closes #549

### Changes are visible to end-users: yes

- Breaking change (forces users to change their own code or config): no
but...

The formatter will now format more files in the users' repository.

The "Shell" run is also slower.

For reference, it runs in about 10 seconds on my laptop in the
https://github.com/Checkmk/checkmk repository. I would say that it is
acceptable. For comparison, formatting our (unfortunately closed-source)
C++ code takes 5 sec.


### Test plan

- Covered by existing test cases: `shfmt` is already supported
- New test cases added: Copied a `hello.sh` to `hello_sh` (no extension)
in the example directory and made sure it's picked up by the formatter.
- Manual testing: I also ran `bazel run //:format` with the current
patch in https://github.com/Checkmk/checkmk and it reformatted files
identified only by their shebang

v1.9.1

Toggle v1.9.1's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
fix: clang-tidy in external modules: use full paths (#628)

Solves #627 

---

### Changes are visible to end-users: yes/no

Should enable the aspect to run on stuff in external modules now.

### Test plan

v1.9.0

Toggle v1.9.0's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
fix: port ruff fixes from aspect-internal silo (#623)

v1.8.0

Toggle v1.8.0's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
chore(deps): update to newer rules_buf (#616)

Seems to improve wall time of running formatter

v1.7.0

Toggle v1.7.0's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
fix: move rules_diff above SNIP

It doesn't need to be in user install instructions.

Fixes #613