Ecosyste.ms: OpenCollective

An open API service for software projects hosted on Open Collective.

github.com/detekt/detekt

Static code analysis for Kotlin
https://github.com/detekt/detekt

Move exclusion out of `Rule` (#6865)

* Move classes to the correct file

In the next commit this function will be used not only for...

a085316617c71620650f2fc408b896d925715e06 authored 12 months ago by Brais Gabín <[email protected]>
Remove all the usages of mockk (#6860)

* Remove all the mocks

* Apply suggestions from code review

* Update detekt-core/src/test/...

109a55d1af523111bec21357071189f428736b19 authored 12 months ago by Nicola Corti <[email protected]>
Add MissingUseCall.kt rule (#6026)

* Add MissingUseCall.kt rule

* Apply suggestions from code review

Co-authored-by: Nicola C...

6a499e37c84b00ee64b959f2e2ce797018bee159 authored 12 months ago by Atul Gupta <[email protected]>
Fis tests (#6870)

e6cc927e98edbbf8ca02428a969aaf04ba6593d2 authored 12 months ago by Brais Gabín <[email protected]>
Align the name of the test code with the name of the production code (#6868)

129a12dbd6a11612c6b0d3a9e142274d1b09afbc authored 12 months ago by Brais Gabín <[email protected]>
Don't use value class for RuleSet.Id (#6861)

* Don't use value class

* Use Poko

c94c72b577741a7768adcc17ebb466f388d31afc authored 12 months ago by Brais Gabín <[email protected]>
Add missing KDoc for Config.parent (#6864)

8d8d6e99af2d16bf85eee460268ee5b557a150e2 authored 12 months ago by Nicola Corti <[email protected]>
Fix exclusion at ruleset level (#6831)

* Simplify code

* Only check the relative path when excluding ruleSets

25c1566db6fad74c3896d80541c33c6e74f24677 authored 12 months ago by Brais Gabín <[email protected]>
Rename core/util/Config.kt to core/util/ConfigExtensions.kt (#6862)

ea2f3ea9f8a0fe686c517a69a33107b07da57d18 authored 12 months ago by Nicola Corti <[email protected]>
Disable the Gradle Daemon on Vercel (#6863)

cba6158d296d07dde8446c0815b352719529512b authored 12 months ago by Nicola Corti <[email protected]>
Create a value class for the id of a `Rule` (#6851)

* Use createIssue

* The report should always report the issue

* Remove unnecessary asserts...

7474fc78d38d9e033d186dc96d8cc385e19a69f9 authored 12 months ago by Brais Gabín <[email protected]>
Add missing tests (#6830)

* Allow to generate files with relative paths

* Inline tests

* Add missing tests

ec7297897640603782b0cbf1e3d8af3248e9bb79 authored 12 months ago by Brais Gabín <[email protected]>
Update docusaurus monorepo to v3.1.0 (#6821)

* Update docusaurus monorepo to v3.1.0

* Fix broken kdoc link

---------

Co-authored-by:...

883e0e30e5792b4c1efc4e6bc4351a612c2710ef authored 12 months ago by renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Simplify code (#6859)

4444f17fa811f515ae94360f57530ce8f21bf3c6 authored 12 months ago by Brais Gabín <[email protected]>
Make fakes more real (#6829)

* Move check up

* Simplify code

* Use more production code to create the fake KtFiles

3c6eee5df6d7cea00055d3bf258393a0b2108812 authored 12 months ago by Brais Gabín <[email protected]>
Remove commaSeparatedPattern (#6820)

* Simplify code

* Remove commaSeparatedPattern from detekt-api

4778c176f82199ecbc5fffa2a3c41bfeb957dae6 authored 12 months ago by Brais Gabín <[email protected]>
Remove List<Finding>.assert() extension function (#6856)

This was only used in a single test class and leads to inconsistent
assertion formatting at the...

325f3b39f6aa82c2c5b43f2f7ac6251c54fe1335 authored 12 months ago by Matthew Haughton <[email protected]>
EmptyKtFile : add ability to ignore package statements (#6790)

* add ability to ignore package statements

* fix code smell

* remove option

* Apply sug...

9abed4fcf590db29e50277de4008cea33b7340ff authored 12 months ago by Matt Groth <[email protected]>
Import custom assertions together with AssertJ assertions (#6857)

This allows assertThat to be used consistently regardless of which set of
assertions is being u...

af4953276960494ffa4f5f6f86cf19d680468d50 authored 12 months ago by Matthew Haughton <[email protected]>
Make the abi a bit cleaner (#6855)

b2a1ea68628612f7f8f3c1e2e5c32db811b144f0 authored 12 months ago by Brais Gabín <[email protected]>
Update actions/upload-artifact digest to 1eb3cb2 (#6852)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

3c827e23c2f245f672463e9b3b4a2b1085897910 authored 12 months ago by renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Update plugin org.gradle.toolchains.foojay-resolver-convention to v0.8.0 (#6853)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

ebeeee1ad2dec5ca9c2506d8ac4322bf62ef1024 authored 12 months ago by renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Create a value class for the id of a `RuleSet` (#6849)

* Don't use Locale.US

* Improve toRunPolicy

* alias RuleSetId to value class RuleSet.Id

...

5e5f958552e553a295f1170433635bdabe447a04 authored 12 months ago by Brais Gabín <[email protected]>
Core handles active rules (#6782)

* Rules doesn't handle if they are active or not

* rename isActive to isActiveOrDefault

--...

430927ce0ef0af7d2934ff4b64e274b01c1f4585 authored 12 months ago by Brais Gabín <[email protected]>
Less verbose tests (#6846)

* Less verbose tests

* Even more quiet

* Simplify code

93535d648b2b891eb72b7508a7f6bbd47da45040 authored 12 months ago by Brais Gabín <[email protected]>
Remove `abstract` from `Rule.issue` (#6794)

* Move issue as first declaration

* Issue

* asdf

* Replace all

Replace this:
: Rule...

d87918b3b27097ff20b354225eef1736a82968b8 authored 12 months ago by Brais Gabín <[email protected]>
Remove `findings` property from `Rule` (#6845)

* Simplify imports

* Don't use Rule.findings on tests

* Don't use visit on tests

* Remove fin...

178d1c8e3da9b1a5a1193e33cb200084fb8f621c authored 12 months ago by Brais Gabín <[email protected]>
Improve error messages for failing assertions (#6847)

* Improve error messages for failing assertions

* Update detekt-rules-coroutines/src/test/kotli...

cf2a5531aa0ee0d40346d096de969ef9784085b1 authored 12 months ago by marschwar <[email protected]>
Inline yamls (#6789)

* Inline yamls

* Inline more yamls

* Inline more yamls

* Inline more yamls

1dea1624dc138559ea881e58f3608744cb700e78 authored 12 months ago by Brais Gabín <[email protected]>
Lookup android flag on ruleset config (#6839)

* Lookup android flag on ruleset config

* fail early on programming or configuration error

829cd34a02d4ddea7444ff057944e3ac67f1600e authored 12 months ago by marschwar <[email protected]>
Remove Context interface (#6843)

There is only a single usage by the Rule class so there is no value having
this extracted into a...

59afa7d4690c93b9ca5de3eb7c0d7f054750b8ad authored 12 months ago by Matthew Haughton <[email protected]>
Remove base rule (#6837)

* Depend on Rule instead of ConfigAware

* Rule receives its own configuration

Rule doesn't...

b52faaa7a53c63aa78e50279372fe94dd899909e authored 12 months ago by Volodymyr Stelmashchuk <[email protected]>
Update dependency com.pinterest.ktlint:ktlint-ruleset-standard to v1.1.1 (#6840)

* Update dependency com.pinterest.ktlint:ktlint-ruleset-standard to v1.1.1

* Add maxLineLengt...

d39300404406defc02737569b43a879226f0789f authored 12 months ago by renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Update github/codeql-action digest to e5f05b8 (#6834)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

122e7d1968320f14861dd270645ba3df441f03a1 authored 12 months ago by renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Update slf4j to v2.0.11 (#6836)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

be42515d1629d84f711b19bbf189565bd60d6bc7 authored 12 months ago by renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Rule receives its own configuration (#6767)

* Depend on Rule instead of ConfigAware

* Rule receives its own configuration

Rule doesn't...

86f4de2773c4a1e6664b8afb5544f2f1a3003495 authored 12 months ago by Brais Gabín <[email protected]>
Simplify tests (#6828)

e8d7b6e63e4825c5aa4859eca209fe79d6a87cc6 authored 12 months ago by Brais Gabín <[email protected]>
Don't use explicit byte-buddy version (#6827)

This was only used in the past as mockk pulled an old version transitively
which didn't support...

94119c16bf2e92c30a2204d7cfeb7fa7d9b55cdb authored 12 months ago by Matthew Haughton <[email protected]>
Update dependency io.mockk:mockk to v1.13.9 (#6826)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

95728961be8d7d6a9710fc5cf96a0e4cc0144d20 authored 12 months ago by renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Fail with meaningful error message if the list elements are not all strings (#6825)

1fdc39b0f98a96404e45110fa1486f52acd1ab44 authored 12 months ago by marschwar <[email protected]>
Align how we declare `Issue` inside the `Rule`s (#6792)

* Align code

* Always use the name of the class as the name of the rule

* Use javaClass.simple...

10155370d812d9c3bc55afd8a209bd7c3d17b851 authored 12 months ago by Brais Gabín <[email protected]>
Fix links (#6824)

c0b3a662de4bcd6ce563b077b142e20702737931 authored 12 months ago by Brais Gabín <[email protected]>
Allow the failure severity to be configured (#6417)

* Fail on issues with error severity only

* Support different kind of failure policies

* C...

cc3b33ed74a64ac61333d62d663c57ad72b96d9a authored 12 months ago by marschwar <[email protected]>
RuleSet provides a Factory of rules intead of the rules themselfs (#6733)

* Extract code

* RuleSet provide a factoty instead of a rule

ef9131f7b64ed1d460106f7c7a01980aaca55738 authored 12 months ago by Brais Gabín <[email protected]>
Add parent config property (#6766)

a9d064887bdd26f36f97a6442cf530b562ebe7f2 authored 12 months ago by Brais Gabín <[email protected]>
Remove support for comma separated strings (#6819)

* Don't smartly cast string to list

* config.exclude passes to be a list of Strings

* Remo...

7c025a7c2bb693c13a13ab404370aa1bf3b14f53 authored 12 months ago by Brais Gabín <[email protected]>
Remove unnecessary functions from detekt-psi-utils (#6823)

* Remove getParentExpressionRemovingParenthesis

There is a single usage in detekt rules which...

cf7c7943cf2aba298bd1776352191766b0fe0a15 authored 12 months ago by Matthew Haughton <[email protected]>
Remove Finding.id (#6774)

* Remove unnecessary asserts

* Remove Finding.id

* fix tests

* Improve Finding kdoc

--------...

e9e0074fca631352b982a55f5efe1419b7885abb authored 12 months ago by Brais Gabín <[email protected]>
Remove default value for parentPath (#6764)

dff9b2664aad7c5ee381f544bf83adc4462f0fad authored 12 months ago by Brais Gabín <[email protected]>
Remove deprecated XML tag support (#6808)

* Remove deprecated XML tag support

* Remove unnecessary test

533740459e02560adefec701972c90105b4bf9d0 authored 12 months ago by Matthew Haughton <[email protected]>
Move RequiresTypeResolution annotation from internal to public API (#6806)

* Move RequiresTypeResolution annotation from internal to public API

* Remove redundant fully...

16cac6c04a3f409bac181c2d1cd568e6b397f137 authored 12 months ago by Matthew Haughton <[email protected]>
Move Configuration annotation from internal to public API (#6805)

74677f3a657d52252d47a8d6c039d90de8966b19 authored 12 months ago by Matthew Haughton <[email protected]>
Move CompilerResources from internal to public API (#6811)

* Move CompilerResources to public API

This type is used in BaseRule so it should be part of ...

c5ebb136e34d41f1b43bb4c14a84a1c63c508a17 authored 12 months ago by Matthew Haughton <[email protected]>
Move ActiveByDefault annotation from internal to public API (#6807)

* Move ActiveByDefault annotation from internal to public API

* Update description of ActiveB...

40fd94040857ef380e67f40fd5ddce2f35d2183d authored 12 months ago by Matthew Haughton <[email protected]>
Make test pass on JVM 8 (#6817)

1425c8510724b2fef94a1813a9ca33c0e9cdb16a authored 12 months ago by Brais Gabín <[email protected]>
ConfigAware doesn't implement Config (#6736)

aa49d9f5dd875794b6c823b9e4bee9f12d2c5246 authored 12 months ago by Brais Gabín <[email protected]>
Fix test at `CorrectableRulesFirstSpec` (#6791)

* Fix CorrectableRulesFirstSpec

* Make the test easier to read

a03e3be950e29663f14bf036f1a1f77014dfe21b authored 12 months ago by Brais Gabín <[email protected]>
Reduce the usage of mockk (#6816)

97f16ff552d0fd8b131d62389118e3588017b472 authored 12 months ago by Brais Gabín <[email protected]>
Move empty config declaration to its callsite (#6809)

This helps tidy the internal package a little.

17253d86f7ccaa316fd259df5f8df6c32cbfef92 authored 12 months ago by Matthew Haughton <[email protected]>
Remove UnstableApi annotation (#6804)

* Stop declaring UnstableApi

* Remove unnecessary OptIn to UnstableApi

* Remove UnstableApi an...

c79491379b25c84682b123d7f72162aa40786be0 authored 12 months ago by Matthew Haughton <[email protected]>
Remove ruleContext property from Rule constructor (#6810)

c9014bd6e690c3497d9a9ea96afd9484bf2c66b8 authored 12 months ago by Matthew Haughton <[email protected]>
Don't use Config.empty on rule initialisation (#6812)

643d2814a0857561e872aa6cb285f3c87b8dd91b authored 12 months ago by Matthew Haughton <[email protected]>
Remove PRIMITIVES from public API (#6813)

84286416c29541eedc52211a99a7ba6b29bccaa3 authored 12 months ago by Matthew Haughton <[email protected]>
fix(deps): update dependency com.android.tools.build:gradle to v8.2.1 (#6803)

* fix(deps): update dependency com.android.tools.build:gradle to v8.2.1

* Update libs.versions....

5280baad7b81c130e8c60f9707233729bf90507d authored almost 1 year ago by renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Improve versioned documentation (#6801)

52c67132b0d76b0ac3c5ad6ab480bf78adb33ef1 authored almost 1 year ago by Brais Gabín <[email protected]>
Exclude FirErrors class from JaCoCo instrumentation to avoid ASM MethodTooLargeException (#6802)

This is required for code coverage to work correctly when building using
K2 compiler.

346950824667ba88221218c2a77c6d964e19e6bc authored almost 1 year ago by Matthew Haughton <[email protected]>
Fix by checking the existence of label in previous statements (#6671)

6496d2a63e746e8d728b783c44fdcb21b009cc2c authored almost 1 year ago by Atul Gupta <[email protected]>
Fix `CastToNullableType` warning on same or parent type (#6677)

44f22f12193f37aa535c23c463812757d34805bd authored almost 1 year ago by Atul Gupta <[email protected]>
Also check `USELESS_ELVIS` in `UnreachableCode` (#6624)

* Also check `USELESS_ELVIS` in `UnreachableCode`

* Remove unnecessary `?: return` from TC co...

3e946a41f89d301e51677c3cea7ec22ff31a170d authored almost 1 year ago by Atul Gupta <[email protected]>
Fix test snippets for K2 (#6799)

In language version 2.0 it is not permitted to use `protected` on a class
that is in a file. As...

caaf3d7c4c1cf42ea0e28a006923f9f0a28628f7 authored almost 1 year ago by Matthew Haughton <[email protected]>
Properly handle nullable type (#6798)

`actual` is a nullable type but the K1 compiler ignores that when
referencing `actual` in funct...

2689cd403f1f91817ef811509e265184c1cd56de authored almost 1 year ago by Matthew Haughton <[email protected]>
fix(deps): update dependency org.assertj:assertj-core to v3.25.1 (#6797)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

04ed825262dc487c05778483de116f37e5bc59a9 authored about 1 year ago by renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Improve documentation (#6793)

1208434a0cbb00b3d347c4a36b0c8cddcbd02490 authored about 1 year ago by Brais Gabín <[email protected]>
fix(deps): update dependency dev.drewhamilton.poko:poko-annotations to v0.15.2 (#6795)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

8dd2994815076e3f8825cc98f5da9cf9a2047df8 authored about 1 year ago by renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Fix #6665 by re-adding a few used toStrings (#6678)

* Re-add toString for FilePath with test
* Re-add toString for Location with test
* Re-add toS...

cd05a8d0edd9e52790b09772f60eb998e988addf authored about 1 year ago by Róbert Papp <[email protected]>
fix(deps): update dependency org.assertj:assertj-core to v3.25.0 (#6787)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

d4629a25a1bfe3270c1b29d584de4bd5a926015b authored about 1 year ago by renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Remove messageOrDescription (#6771)

* Always use the same default message

* Don't create findings without message on tests

* M...

a3df60af69a9a99ee5b8cfba77e3f8addf5b479e authored about 1 year ago by Brais Gabín <[email protected]>
fix(deps): update dependency clsx to v2.1.0 (#6784)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

4dc21826ddf76ae7e11ff13a0facaf50a76230b3 authored about 1 year ago by renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Properly declare task input for processTestResources task (#6785)

1ef157aac527dfdec893cbcfe952b720aa4e2f10 authored about 1 year ago by Matthew Haughton <[email protected]>
fix(deps): update slf4j to v2.0.10 (#6783)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

8cd4b188523d771bc54caeeabc47c7e8ebb972f3 authored about 1 year ago by renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Improve Analyzer tests (#6781)

9057684f33468d0a25cd976f1e33150cb3d39afa authored about 1 year ago by Brais Gabín <[email protected]>
Remove input and output file registration for generateDocumentation task (#6763)

This task has caching disabled. When enabling with outputs.cacheIf {true}
there is an error abou...

7d8f3856de7e52d87f094ae42c630edc39a1bd85 authored about 1 year ago by Matthew Haughton <[email protected]>
Mandatory config on Rule (#6734)

* Make config no optional on Rule

* Remove redundant val

58698212fcbf91ee1060801ebb23ce02880db9d9 authored about 1 year ago by Brais Gabín <[email protected]>
Use entries instead of values() (#6769)

0c113d69dbebe0bf48d50e35fe34f5e387588264 authored about 1 year ago by Brais Gabín <[email protected]>
Remove unnecessary tests (#6777)

d43a77d40b0a623f857a61d2cdf13062feee077d authored about 1 year ago by Brais Gabín <[email protected]>
Add description for ForbiddenPublicDataClass (#6770)

a2e224c6ba2c011d12b5a5f49d9157e6db30899b authored about 1 year ago by Brais Gabín <[email protected]>
Remove BaselineProvider.id() (#6773)

0da08c050e7aa1cb55ff01721f5c73ff9421396a authored about 1 year ago by Brais Gabín <[email protected]>
Reduce the usage of Mockk (#6772)

* Add default value for createEntitiy(path)

* Don't use mockk when it is not needed

* Remove u...

889ee02102cb36efc2a12de07fdb35ef8950f981 authored about 1 year ago by Brais Gabín <[email protected]>
Add `BigDecimal`'s double constructor to `ForbiddenMethodCall`'s default configuration (#6744)

09024e26fa6e8d774efd5b48e3f5c0dad6ca1d31 authored about 1 year ago by Mark Vadeika <[email protected]>
Use Locale.ROOT instead of US (#6765)

30f2d08b5c899241899af9bf35f572f08816803f authored about 1 year ago by Brais Gabín <[email protected]>
Remove outdated documentation (#6768)

08148f4dc865cb9cd3d72d0b089c5ca67f671261 authored about 1 year ago by Brais Gabín <[email protected]>
Fix cli tests (#6776)

36775c7e999dcd21bf955012ade37f6d74c9fd7f authored about 1 year ago by Brais Gabín <[email protected]>
Unblock snapshots by updating the base plugin id (#6779)

3324a9367ac53bfd6982ab2b6a48a3a4bf2c7820 authored about 1 year ago by Nicola Corti <[email protected]>
Remove never used function (#6775)

6bf2d18a2041189493b0780c31b9c05013d0bd8a authored about 1 year ago by Brais Gabín <[email protected]>
Remove deprecated code (#6726)

b8c8dce4bda5e8fbf36c04659c8546b640b30e49 authored about 1 year ago by Brais Gabín <[email protected]>
Remove RuleSet.visitFile (#6730)

f3919a608712716f43675c5985318617d17fb6b0 authored about 1 year ago by Brais Gabín <[email protected]>
Simplfy sarirf code (#6729)

* Simplify Sarif

* Use Root locale insted of US

d98ccadce6246ccc873d176cc4ad86b4b48f1c59 authored about 1 year ago by Brais Gabín <[email protected]>
Fix too generic tests (#6724)

* Simplify tests

* Define the code in the test

* Align format of tests

* Don't depend o...

2c18ee1a649f7e23b6c7d53ccbcf0c9802363ea1 authored about 1 year ago by Brais Gabín <[email protected]>
Move APIs to a dedicated submenu and fix links (#6757)

3683671dddb6f58ed755162edcd2b7319cf967e6 authored about 1 year ago by Nicola Corti <[email protected]>
fix(deps): update dependency prism-react-renderer to v2.3.1 (#6756)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

8f5fcf807da2abf993f0d4990f594c3d6c5ba563 authored about 1 year ago by renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>