Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Linting - API #2149

Merged
merged 35 commits into from
Oct 21, 2024
Merged
Changes from 1 commit
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
9afa6dd
Handy tool for making encoding formats that don't have any forbidden …
nedtwigg May 30, 2024
319fc79
Revert "Handy tool for making encoding formats that don't have any fo…
nedtwigg May 30, 2024
777fd5c
Add a `Lint` class, along with `ShortcutException` for sending them.
nedtwigg May 30, 2024
322d5bf
Add a `lint` method to the core interfaces: `Formatter[Step|Func]`
nedtwigg May 30, 2024
c9f52d5
Pipe linting through the core FormatterStep implementations.
nedtwigg May 30, 2024
b57bf24
Formatter can now capture exceptions per-formatter, rethrows if you d…
nedtwigg May 30, 2024
a94dce9
Pipe the lints through `FenceStep`, preliminary.
nedtwigg May 30, 2024
c224929
Remove all of `FormatExceptionPolicy` except `Strict`, which remains …
nedtwigg May 30, 2024
b01e9e9
Rename `ExceptionPerStep` to `ValuePerStep`, and bring `Formatter` cl…
nedtwigg May 30, 2024
fe92d05
Update `Formatter` and `ValuePerStep` so that something (perhaps null…
nedtwigg May 31, 2024
f3be100
Introduce LintState which efficiently reads lint data from both `form…
nedtwigg May 31, 2024
5c42457
Restore the "legacy" error printing for both `Formatter` and `DirtySt…
nedtwigg May 31, 2024
d467545
Fix spotbugs.
nedtwigg May 31, 2024
700114f
Add Selfie, and configure it to not use triple quote literals.
nedtwigg Jun 4, 2024
ffc911e
Add a way to test for lints, and use that to bring back FenceStepTest…
nedtwigg Jun 4, 2024
c2fe9c6
Merge branch 'feat/prepare-for-lint-take-2' into feat/lint-take-2
nedtwigg Jun 4, 2024
6bbe556
Deal with `DirtyState.Calculation` is gone.
nedtwigg Jun 4, 2024
aa769d0
Merge branch 'feat/prepare-for-lint-take-2' into feat/lint-take-2
nedtwigg Oct 15, 2024
d08f442
Two maven tests which fail because errors are getting swallowed as li…
nedtwigg Oct 16, 2024
53e31f5
Some gradle tests which fail because errors are getting swallowed as …
nedtwigg Oct 16, 2024
9af4880
DirtyState now uses `LintPolicy.legacyBehavior` unless you passed in …
nedtwigg Oct 16, 2024
dfca397
Rework SpotlessTask to use `LintState` so that we can know which step…
nedtwigg Oct 17, 2024
29cba64
We don't need to move lints to their own place *yet*.
nedtwigg Oct 17, 2024
43cd94b
Make an explicit constant for a lint at an undefined line.
nedtwigg Oct 17, 2024
7bf6b1b
Merge branch 'main' into feat/lint-take-2
nedtwigg Oct 17, 2024
6619a76
Fix windows.
nedtwigg Oct 17, 2024
a9bd0a6
Update changelog.
nedtwigg Oct 17, 2024
6c480d8
Add info to CONTRIBUTING
nedtwigg Oct 17, 2024
2fb5a16
Merge branch 'main' into feat/lint-take-2
nedtwigg Oct 17, 2024
ae31ea5
Setup selfie.
nedtwigg Oct 20, 2024
8300fda
Fix ambiguities in the `Lint` class around treating it as data vs exc…
nedtwigg Oct 20, 2024
9658d41
Improve LintState's toString.
nedtwigg Oct 20, 2024
d00a4ab
Adapt gradle test for the API.
nedtwigg Oct 20, 2024
195bc16
Remove `testResourceExceptionMsg` and replace with `expectLintsOfReso…
nedtwigg Oct 20, 2024
14cbc52
Merge branch 'main' into feat/lint-take-2
nedtwigg Oct 20, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'main' into feat/lint-take-2
nedtwigg authored Oct 17, 2024
commit 2fb5a16014b6ee77f74a32b3e21474ea0914797c
2 changes: 2 additions & 0 deletions CHANGES.md
Original file line number Diff line number Diff line change
@@ -19,6 +19,8 @@ We adhere to the [keepachangelog](https://keepachangelog.com/en/1.0.0/) format (
### Changes
* **BREAKING** Moved `PaddedCell.DirtyState` to its own top-level class with new methods. ([#2148](https://github.com/diffplug/spotless/pull/2148))
* **BREAKING** Removed `isClean`, `applyTo`, and `applyToAndReturnResultIfDirty` from `Formatter` because users should instead use `DirtyState`.
### Fixed
* `ktlint` steps now read from the `string` instead of the `file` so they don't clobber earlier steps. (fixes [#1599](https://github.com/diffplug/spotless/issues/1599))

## [3.0.0.BETA3] - 2024-10-15
### Added
You are viewing a condensed version of this merge commit. You can view the full changes here.