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

Handle corner case of multiple inheritance in UnnecessaryAbstractClass rule #1

Merged
merged 65 commits into from Jul 10, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
65 commits
Select commit Hold shift + click to select a range
e33878c
Drop dependency-analysis plugin (#4946)
3flex Jun 11, 2022
d06b109
Gradle Enterprise Trial (#4763)
chao2zhang Jun 12, 2022
44d0e81
Pin Github Actions SHAs (#4938)
renovate[bot] Jun 13, 2022
d3cdbb6
Enable GE build scan publishing for all workflows (#4947)
3flex Jun 13, 2022
43e0a4a
Remove Sonarqube Gradle Plugin (#4950)
cortinico Jun 13, 2022
1d47fd3
Support disabling config validation via tooling spec (#4937)
arturbosch Jun 13, 2022
43ea42c
Fix a typo in GRADLE_CACHE_PASSWORD credentials and enable remote cac…
chao2zhang Jun 13, 2022
5d49409
Add Markdown report (#4858)
VitalyVPinchuk Jun 14, 2022
a10e957
Improve readme (#4914)
BraisGabin Jun 14, 2022
feec74c
Update plugin binaryCompatibilityValidator to v0.10.1 (#4954)
renovate[bot] Jun 14, 2022
195d15a
Update react monorepo to v18.2.0 (#4958)
renovate[bot] Jun 14, 2022
e8e1f35
Set strict dependency on tested Kotlin compiler version (#4822)
3flex Jun 15, 2022
8b62ffc
Update dependency com.github.breadmoirai:github-release to v2.4.1 (#4…
renovate[bot] Jun 17, 2022
b06276a
Fix rule code samples to be valid Kotlin code (#4969)
3flex Jun 19, 2022
06ee239
Use plain ASCII output in standard reports (#4968)
3flex Jun 19, 2022
ecd28cf
Simplify signing config (#4971)
3flex Jun 20, 2022
bd262c6
Update dependency com.gradle.plugin-publish to 1.0.0-rc-3 (#4970)
3flex Jun 20, 2022
bdec78d
Update gradle/gradle-build-action digest to 67421db (#4977)
renovate[bot] Jun 20, 2022
75f38dc
Update dependency org.jetbrains.kotlinx:kotlinx-coroutines-core to v1…
renovate[bot] Jun 22, 2022
7832eef
Give a better error message for --jvm-target (#4978)
yogurtearl Jun 22, 2022
0e67d14
Add instuctions for CLI installation using Scoop (#4984)
Jun 22, 2022
89f6ec1
Add CascadingCallWrapping style rule (#4979)
dzirbel Jun 22, 2022
4979e37
Do not allow junit assertions (#4987)
marschwar Jun 22, 2022
8dc783a
Measure flakyness (#4742)
BraisGabin Jun 22, 2022
4742842
Fix false-negative for CanBeNonNullable (#4993)
VitalyVPinchuk Jun 24, 2022
3d0f215
Update actions/setup-java digest to 16cca54 (#4996)
renovate[bot] Jun 25, 2022
4441bc4
Update github/codeql-action digest to 41a4ada (#4981)
renovate[bot] Jun 25, 2022
3b7cae1
Support markdown report in Gradle plugin (#4995)
VitalyVPinchuk Jun 27, 2022
5170077
Address TextLocation for Wrapping (#4998)
chao2zhang Jun 27, 2022
2b068e0
Fix Signatures.kt:buildFunctionSignature (#4961)
VitalyVPinchuk Jun 27, 2022
0c879a7
Update dependency org.jetbrains.dokka to v1.7.0 (#4974)
renovate[bot] Jun 27, 2022
c8eb24c
UnnecessaryApply: fix false negative for assignment (#4948)
dzirbel Jun 27, 2022
5ff82e1
Setup a Remark plugin to replace [detekt_version] with the current ve…
cortinico Jun 27, 2022
5d6ee2f
NamedArguments: don't count trailing lambda argument (#5002)
t-kameyama Jun 28, 2022
29158af
Show finding at declaration name instead of the whole declaration (#5…
arturbosch Jun 28, 2022
1e696fd
Add MaxChainedCallsOnSameLine style rule (#4985)
dzirbel Jun 28, 2022
5c39d7d
Loading a specific resource from a module must use class from module …
arturbosch Jun 29, 2022
0a00c2a
Update github/codeql-action digest to 3f62b75 (#5007)
renovate[bot] Jun 29, 2022
bea15be
Update plugin pluginPublishing to v1.0.0 (#5010)
renovate[bot] Jun 29, 2022
49c6d88
Allow constants to be skipped for BooleanPropertyNaming rule (#5006)
amitdash291 Jun 29, 2022
58d4b8e
Prepare Detekt 1.21.0-RC2 (#5013)
cortinico Jun 30, 2022
292372e
Fix CLI link that is incorrect (#5016)
jlengrand Jun 30, 2022
246ccd8
Removes documentation formatting issue (#5017)
jlengrand Jun 30, 2022
cf8bd5e
Add test for ForbiddenMethodCall with getters (#5018)
schalkms Jul 1, 2022
b037183
Update dependency prism-react-renderer to v1.3.5 (#5019)
renovate[bot] Jul 1, 2022
fbb913b
Update JamesIves/github-pages-deploy-action digest to e6d003d (#5023)
renovate[bot] Jul 1, 2022
b49aea7
Add endColumn/endLine to SARIF region (#5011)
VitalyVPinchuk Jul 1, 2022
1322deb
Fix type resolution link in Contributing.md (#5027)
amitdash291 Jul 1, 2022
ddcf77d
Fix ReturnCount debt and refactor code (#5026)
amitdash291 Jul 1, 2022
6edf6bb
#5014 Fix MaxChainedCallsOnSameLine false positives (#5020)
dzirbel Jul 2, 2022
c953140
Update dependency clsx to v1.2.0 (#5031)
renovate[bot] Jul 5, 2022
aa2a1bf
MaxChainedCallsOnSameLine: don't count package references as chained …
t-kameyama Jul 5, 2022
08a196a
Update dependency clsx to v1.2.1 (#5039)
renovate[bot] Jul 6, 2022
2add9e7
Add 'Revved up by Gradle Enterprise' badge (#5012)
clayburn Jul 6, 2022
1a6d059
Fix `ComplexMethod` debt and refactor code (#5029)
amitdash291 Jul 6, 2022
46aa36f
Only recommend using index accessors for Java classes that are known …
Whathecode Jul 6, 2022
b6448e3
Xml Report Merger now merges duplicate smells across input report fil…
timothyolt Jul 7, 2022
6638b2f
Add ending line and column to Location.kt (#5032)
VitalyVPinchuk Jul 7, 2022
8940fb6
Enable strict Kotlin DSL precompiled script plugins accessors generat…
3flex Jul 8, 2022
7cdd6ca
Drop redundant workaround (#5057)
3flex Jul 9, 2022
8903b2e
Improve extensions.doc format with admonitions (#5055)
schalkms Jul 9, 2022
2e6f16b
Update ktlint links from website to readme (#5056)
schalkms Jul 9, 2022
b643772
Update docusaurus monorepo to v2.0.0-beta.22 (#5050)
renovate[bot] Jul 9, 2022
675c48c
Merge remote-tracking branch 'upstream/main' into fix-4753
amitdash291 Jul 9, 2022
69b3fd3
Handle corner case of multiple inheritance in UnnecessaryAbstractClas…
amitdash291 Jul 9, 2022
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/CONTRIBUTING.md
Expand Up @@ -64,7 +64,7 @@ Rules annotated with `@ActiveByDefault` will be marked as active in the `default
Generally, this will not be the case for new rules.

A rule that requires type resolution must be marked with `@RequiresTypeResolution`.
See [the type resolution wiki page](../docs/pages/gettingstarted/type-resolution.md) for
See [the type resolution wiki page](../website/docs/gettingstarted/type-resolution.md) for
more detail on this topic.

The rule defined above will translate to a rule entry in the `default-detekt-config.yml`:
Expand Down
11 changes: 7 additions & 4 deletions .github/workflows/codecoverage.yaml
Expand Up @@ -8,6 +8,9 @@ on:
branches:
- '**'

env:
GRADLE_ENTERPRISE_ACCESS_KEY: ${{ secrets.GRADLE_ENTERPRISE_ACCESS_KEY }}

permissions:
contents: read

Expand All @@ -17,19 +20,19 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout Repo
uses: actions/checkout@v3
uses: actions/checkout@2541b1294d2704b0964813337f33b291d3f8596b # tag=v3

- name: Setup Java
uses: actions/setup-java@v3
uses: actions/setup-java@16cca5479d7c6b6843f6a6515640ba33c6501543 # tag=v3
with:
java-version: 17
distribution: 'temurin'

- name: Generate Coverage Report
uses: gradle/gradle-build-action@v2
uses: gradle/gradle-build-action@67421db6bd0bf253fb4bd25b31ebb98943c375e1 # tag=v2
with:
arguments: jacocoMergedReport

- name: Publish Coverage
if: success()
uses: codecov/codecov-action@v3
uses: codecov/codecov-action@81cd2dc8148241f03f5839d295e000b8f761e378 # tag=v3
11 changes: 7 additions & 4 deletions .github/workflows/deploy-snapshot.yaml
Expand Up @@ -5,6 +5,9 @@ on:
branches:
- main

env:
GRADLE_ENTERPRISE_ACCESS_KEY: ${{ secrets.GRADLE_ENTERPRISE_ACCESS_KEY }}

permissions:
contents: read

Expand All @@ -14,21 +17,21 @@ jobs:
if: github.repository == 'detekt/detekt' && !contains(github.event.head_commit.message, 'ci skip')
steps:
- name: Checkout Repo
uses: actions/checkout@v3
uses: actions/checkout@2541b1294d2704b0964813337f33b291d3f8596b # tag=v3

- name: Setup Java
uses: actions/setup-java@v3
uses: actions/setup-java@16cca5479d7c6b6843f6a6515640ba33c6501543 # tag=v3
with:
java-version: 17
distribution: 'temurin'

- name: Build detekt
uses: gradle/gradle-build-action@v2
uses: gradle/gradle-build-action@67421db6bd0bf253fb4bd25b31ebb98943c375e1 # tag=v2
with:
arguments: build

- name: Deploy Snapshot
uses: gradle/gradle-build-action@v2
uses: gradle/gradle-build-action@67421db6bd0bf253fb4bd25b31ebb98943c375e1 # tag=v2
env:
ORG_GRADLE_PROJECT_SIGNING_KEY: ${{ secrets.ORG_GRADLE_PROJECT_SIGNING_KEY }}
ORG_GRADLE_PROJECT_SIGNING_PWD: ${{ secrets.ORG_GRADLE_PROJECT_SIGNING_PWD }}
Expand Down
16 changes: 9 additions & 7 deletions .github/workflows/detekt-with-type-resolution.yaml
Expand Up @@ -10,6 +10,8 @@ on:

env:
GRADLE_ENTERPRISE_ACCESS_KEY: ${{ secrets.GRADLE_ENTERPRISE_ACCESS_KEY }}
GRADLE_CACHE_USERNAME: ${{ secrets.GRADLE_CACHE_USERNAME }}
GRADLE_CACHE_PASSWORD: ${{ secrets.GRADLE_CACHE_PASSWORD }}

permissions:
contents: read
Expand All @@ -23,21 +25,21 @@ jobs:
if: ${{ !contains(github.event.head_commit.message, 'ci skip') }}
steps:
- name: Checkout Repo
uses: actions/checkout@v3
uses: actions/checkout@2541b1294d2704b0964813337f33b291d3f8596b # tag=v3

- name: Setup Java
uses: actions/setup-java@v3
uses: actions/setup-java@16cca5479d7c6b6843f6a6515640ba33c6501543 # tag=v3
with:
java-version: 17
distribution: 'temurin'

- name: Run detekt-cli with argsfile
uses: gradle/gradle-build-action@v2
uses: gradle/gradle-build-action@67421db6bd0bf253fb4bd25b31ebb98943c375e1 # tag=v2
with:
arguments: :detekt-cli:runWithArgsFile

- name: Upload SARIF to Github using the upload-sarif action
uses: github/codeql-action/upload-sarif@v2
uses: github/codeql-action/upload-sarif@3f62b754e23e0dd60f91b744033e1dc1654c0ec6 # tag=v2
if: ${{ always() }}
with:
sarif_file: build/detekt-report.sarif
Expand All @@ -48,14 +50,14 @@ jobs:
if: ${{ !contains(github.event.head_commit.message, 'ci skip') }}
steps:
- name: Checkout Repo
uses: actions/checkout@v3
uses: actions/checkout@2541b1294d2704b0964813337f33b291d3f8596b # tag=v3

- name: Setup Java
uses: actions/setup-java@v3
uses: actions/setup-java@16cca5479d7c6b6843f6a6515640ba33c6501543 # tag=v3
with:
java-version: 17
distribution: 'temurin'
- name: Run analysis
uses: gradle/gradle-build-action@v2
uses: gradle/gradle-build-action@67421db6bd0bf253fb4bd25b31ebb98943c375e1 # tag=v2
with:
arguments: detektMain detektTest
4 changes: 2 additions & 2 deletions .github/workflows/fossascan.yaml
Expand Up @@ -14,9 +14,9 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout Repo
uses: actions/checkout@v3
uses: actions/checkout@2541b1294d2704b0964813337f33b291d3f8596b # tag=v3

- name: Run FOSSA Scan
uses: fossas/fossa-action@v1
uses: fossas/fossa-action@30055fd3dbc35d6ca57da934b5d8ac213a7871b4 # tag=v1
with:
api-key: ${{ secrets.FOSSA_API_KEY }}
4 changes: 2 additions & 2 deletions .github/workflows/gradle-wrapper-validation.yml
Expand Up @@ -16,6 +16,6 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout latest code
uses: actions/checkout@v3
uses: actions/checkout@2541b1294d2704b0964813337f33b291d3f8596b # tag=v3
- name: Validate Gradle Wrapper
uses: gradle/wrapper-validation-action@v1
uses: gradle/wrapper-validation-action@e6e38bacfdf1a337459f332974bb2327a31aaf4b # tag=v1
2 changes: 1 addition & 1 deletion .github/workflows/labeler.yml
Expand Up @@ -9,6 +9,6 @@ jobs:
pull-requests: write
runs-on: ubuntu-latest
steps:
- uses: actions/labeler@v4
- uses: actions/labeler@9fd24f1f9d6ceb64ba34d181b329ee72f99978a0 # tag=v4
with:
repo-token: "${{ secrets.GITHUB_TOKEN }}"
44 changes: 31 additions & 13 deletions .github/workflows/pre-merge.yaml
Expand Up @@ -10,6 +10,8 @@ on:

env:
GRADLE_ENTERPRISE_ACCESS_KEY: ${{ secrets.GRADLE_ENTERPRISE_ACCESS_KEY }}
GRADLE_CACHE_USERNAME: ${{ secrets.GRADLE_CACHE_USERNAME }}
GRADLE_CACHE_PASSWORD: ${{ secrets.GRADLE_CACHE_PASSWORD }}

permissions:
contents: read
Expand All @@ -28,31 +30,31 @@ jobs:
runs-on: ${{ matrix.os }}
steps:
- name: Checkout Repo
uses: actions/checkout@v3
uses: actions/checkout@2541b1294d2704b0964813337f33b291d3f8596b # tag=v3
- name: Setup Java
uses: actions/setup-java@v3
uses: actions/setup-java@16cca5479d7c6b6843f6a6515640ba33c6501543 # tag=v3
with:
java-version: ${{ matrix.jdk }}
distribution: 'temurin'
- name: Build detekt
uses: gradle/gradle-build-action@v2
uses: gradle/gradle-build-action@67421db6bd0bf253fb4bd25b31ebb98943c375e1 # tag=v2
with:
arguments: build -x detekt
- uses: actions/upload-artifact@v3
- uses: actions/upload-artifact@3cea5372237819ed00197afe530f5a7ea3e805c8 # tag=v3
with:
name: heap-dump
path: '**.hprof'
if-no-files-found: ignore
- name: Run detekt-cli --help
uses: gradle/gradle-build-action@v2
uses: gradle/gradle-build-action@67421db6bd0bf253fb4bd25b31ebb98943c375e1 # tag=v2
with:
arguments: :detekt-cli:runWithHelpFlag
- name: Run detekt-cli with argsfile
uses: gradle/gradle-build-action@v2
uses: gradle/gradle-build-action@67421db6bd0bf253fb4bd25b31ebb98943c375e1 # tag=v2
with:
arguments: :detekt-cli:runWithArgsFile
- name: Try to publish to Maven Local
uses: gradle/gradle-build-action@v2
uses: gradle/gradle-build-action@67421db6bd0bf253fb4bd25b31ebb98943c375e1 # tag=v2
with:
arguments: publishToMavenLocal

Expand All @@ -61,14 +63,14 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout Repo
uses: actions/checkout@v3
uses: actions/checkout@2541b1294d2704b0964813337f33b291d3f8596b # tag=v3
- name: Setup Java
uses: actions/setup-java@v3
uses: actions/setup-java@16cca5479d7c6b6843f6a6515640ba33c6501543 # tag=v3
with:
java-version: 17
distribution: 'temurin'
- name: Verify Generated Detekt Config File
uses: gradle/gradle-build-action@v2
uses: gradle/gradle-build-action@67421db6bd0bf253fb4bd25b31ebb98943c375e1 # tag=v2
with:
arguments: verifyGeneratorOutput

Expand All @@ -77,13 +79,29 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout Repo
uses: actions/checkout@v3
uses: actions/checkout@2541b1294d2704b0964813337f33b291d3f8596b # tag=v3
- name: Setup Java
uses: actions/setup-java@v3
uses: actions/setup-java@16cca5479d7c6b6843f6a6515640ba33c6501543 # tag=v3
with:
java-version: 17
distribution: 'temurin'
- name: Build and compile test snippets
uses: gradle/gradle-build-action@v2
uses: gradle/gradle-build-action@67421db6bd0bf253fb4bd25b31ebb98943c375e1 # tag=v2
with:
arguments: test -Pcompile-test-snippets=true

warnings-as-errors:
if: ${{ !contains(github.event.head_commit.message, 'ci skip') }}
runs-on: ubuntu-latest
steps:
- name: Checkout Repo
uses: actions/checkout@2541b1294d2704b0964813337f33b291d3f8596b # tag=v3
- name: Setup Java
uses: actions/setup-java@16cca5479d7c6b6843f6a6515640ba33c6501543 # tag=v3
with:
java-version: 17
distribution: 'temurin'
- name: Run with allWarningsAsErrors
uses: gradle/gradle-build-action@67421db6bd0bf253fb4bd25b31ebb98943c375e1 # tag=v2
with:
arguments: build -x detekt -PwarningsAsErrors=true
2 changes: 1 addition & 1 deletion .github/workflows/stale.yaml
Expand Up @@ -10,7 +10,7 @@ jobs:
issues: write
pull-requests: write
steps:
- uses: actions/stale@v5
- uses: actions/stale@3cc123766321e9f15a6676375c154ccffb12a358 # tag=v5
with:
repo-token: ${{ secrets.GITHUB_TOKEN }}
days-before-stale: 90
Expand Down
13 changes: 8 additions & 5 deletions .github/workflows/website.yaml
Expand Up @@ -8,6 +8,9 @@ on:
branches:
- '**'

env:
GRADLE_ENTERPRISE_ACCESS_KEY: ${{ secrets.GRADLE_ENTERPRISE_ACCESS_KEY }}

permissions:
contents: write # for JamesIves/github-pages-deploy-action to push changes in repo

Expand All @@ -17,23 +20,23 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout Repo
uses: actions/checkout@v3
uses: actions/checkout@2541b1294d2704b0964813337f33b291d3f8596b # tag=v3

- name: Setup Java
uses: actions/setup-java@v3
uses: actions/setup-java@16cca5479d7c6b6843f6a6515640ba33c6501543 # tag=v3
with:
java-version: 17
distribution: 'temurin'

- name: Setup Node
uses: actions/setup-node@v3
uses: actions/setup-node@eeb10cff27034e7acf239c5d29f62154018672fd # tag=v3
with:
node-version: '16'
cache: 'yarn'
cache-dependency-path: 'website/yarn.lock'

- name: Build Detekt Documentation
uses: gradle/gradle-build-action@v2
uses: gradle/gradle-build-action@67421db6bd0bf253fb4bd25b31ebb98943c375e1 # tag=v2
with:
arguments: :detekt-generator:generateDocumentation

Expand All @@ -46,7 +49,7 @@ jobs:
run: yarn build

- name: Deploy Github Pages (only on main)
uses: JamesIves/github-pages-deploy-action@v4
uses: JamesIves/github-pages-deploy-action@e6d003d0839927f5a4b998bfd92ed8e448fde37a # tag=v4
if: ${{ github.event_name == 'push' && github.ref == 'refs/heads/main' }}
with:
branch: gh-pages
Expand Down