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

Adding ESLint as formatter step #1453

Merged
merged 39 commits into from
Jan 10, 2023
Merged
Changes from 1 commit
Commits
Show all changes
39 commits
Select commit Hold shift + click to select a range
4959227
extract base serve code
simschla Nov 28, 2022
c70bd34
upgrade express to latest
simschla Nov 28, 2022
3884e14
extract common rest code on client side
simschla Dec 3, 2022
c20d3bb
add "eslint --fix" as formatter
simschla Dec 3, 2022
4f5e4d3
eslint: add support for xo standard typescript ruleset
simschla Dec 7, 2022
9633967
eslint: add support for standard-with-typescript ruleset
simschla Dec 9, 2022
7022b7f
eslint: add unit tests, cleanup code
simschla Dec 15, 2022
a2a5502
eslint: reduce required options
simschla Dec 15, 2022
35b2b7c
eslint: remove unneeded configuration
simschla Dec 16, 2022
016dd3a
eslint: test supplying inline config
simschla Dec 16, 2022
a9f89d8
eslint: adding javascript formatting (part 1)
simschla Dec 16, 2022
4efc19b
eslint: introduce separate javascript extension
simschla Dec 18, 2022
0206246
eslint: adding tests for js
simschla Dec 20, 2022
393a4c4
eslint: bumping version numbers
simschla Dec 20, 2022
8dc740e
eslint: adapt to extended api on maven side
simschla Dec 20, 2022
cb3dd55
eslint: speedup npm install
simschla Dec 21, 2022
e12c5ef
eslint: speedup npmrc tests
simschla Dec 21, 2022
218db79
eslint: extract base properties
simschla Dec 21, 2022
31462a2
eslint: initial maven support
simschla Dec 22, 2022
dd5ae78
eslint: tests for maven integration into typescript
simschla Dec 22, 2022
7d2e5ad
eslint: adding maven javascript tests
simschla Dec 23, 2022
1d164d3
eslint: also test custom devDependencies case
simschla Dec 23, 2022
3f37053
eslint: adding readme (common and plugin-gradle)
simschla Jan 4, 2023
bb843ce
eslint: apply spotless
simschla Jan 4, 2023
508dd69
bumping default prettier version
simschla Jan 4, 2023
67a0a05
eslint: update readme for maven
simschla Jan 4, 2023
e07082a
eslint: unify naming in documentation
simschla Jan 4, 2023
2943c66
eslint: adapt to api changes in TestResource
simschla Jan 4, 2023
ce21de7
eslint: adapt PR number
simschla Jan 4, 2023
3a1d43b
eslint: add default version bump to CHANGES
simschla Jan 4, 2023
b4f76b4
eslint: adapt maven code to match documentation
simschla Jan 4, 2023
4598319
eslint: spotlessApply
simschla Jan 4, 2023
3540242
eslint: reduce log-noise, cleanup error codes
simschla Jan 5, 2023
98c023b
eslint: spotbugs fix
simschla Jan 5, 2023
9a0932e
Merge branch 'main' of https://github.com/diffplug/spotless into feat…
simschla Jan 9, 2023
d4d91cb
eslint: remove direct api support for styleguides
simschla Jan 10, 2023
7673e22
Merge branch 'main' of https://github.com/diffplug/spotless into feat…
simschla Jan 10, 2023
6577e29
eslint: fix anchor navigation
simschla Jan 10, 2023
0227e89
eslint: cleanup doc
simschla Jan 10, 2023
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
Original file line number Diff line number Diff line change
@@ -19,6 +19,8 @@
import java.util.Map;
import java.util.TreeMap;

import com.diffplug.spotless.ResourceHarness;

import org.junit.jupiter.api.Nested;
import org.junit.jupiter.api.Test;
import org.junit.jupiter.params.ParameterizedTest;
@@ -73,8 +75,8 @@ void formattingUsingRulesetsFile(String ruleSetName) throws Exception {
npmPathResolver(),
new EslintConfig(eslintRc, null));

try (StepHarnessWithFile stepHarness = StepHarnessWithFile.forStep(formatterStep)) {
stepHarness.testResource(dirtyFileFile, dirtyFile, cleanFile);
try (StepHarnessWithFile stepHarness = StepHarnessWithFile.forStep(this, formatterStep)) {
stepHarness.test(dirtyFileFile, ResourceHarness.getTestResource(dirtyFile), ResourceHarness.getTestResource(cleanFile));
}
}
}
@@ -116,8 +118,8 @@ void formattingUsingRulesetsFile(String ruleSetName) throws Exception {
npmPathResolver(),
new EslintTypescriptConfig(eslintRc, null, tsconfigFile));

try (StepHarnessWithFile stepHarness = StepHarnessWithFile.forStep(formatterStep)) {
stepHarness.testResource(dirtyFileFile, dirtyFile, cleanFile);
try (StepHarnessWithFile stepHarness = StepHarnessWithFile.forStep(this, formatterStep)) {
stepHarness.test(dirtyFileFile, ResourceHarness.getTestResource(dirtyFile), ResourceHarness.getTestResource(cleanFile));
}
}
}
@@ -173,8 +175,8 @@ void formattingUsingInlineXoConfig() throws Exception {
npmPathResolver(),
new EslintTypescriptConfig(null, esLintConfig, tsconfigFile));

try (StepHarnessWithFile stepHarness = StepHarnessWithFile.forStep(formatterStep)) {
stepHarness.testResource(dirtyFileFile, dirtyFile, cleanFile);
try (StepHarnessWithFile stepHarness = StepHarnessWithFile.forStep(this, formatterStep)) {
stepHarness.test(dirtyFileFile, ResourceHarness.getTestResource(dirtyFile), ResourceHarness.getTestResource(cleanFile));
}
}
}