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

Detekt Configuration Schema 1.22.0 #2623

Merged
merged 14 commits into from Nov 30, 2022

Conversation

TWiStErRob
Copy link
Contributor

@TWiStErRob TWiStErRob commented Nov 30, 2022

 * detekt\detekt-core\src\main\resources\default-detekt-config.yml
 * gradlew :detektGenerateConfig output on v1.22.0 tag
via https://onlineyamltools.com/convert-yaml-to-json.
through https://www.liquid-technologies.com/online-json-to-schema-converter
Options: array rules = allow anything, make required = false
…ding-json-schema/structuring.html#defs

and https://stackoverflow.com/a/52579526/253468

Search (regex, spaces matter):
```
          "properties": \{
            "active": \{
              "type": "boolean"
            },?
```
Replace (regex):
```
          "allOf": [
            {
              "\$ref": "#/\$defs/ruleProperties"
            }
          ],
          "properties": {
```

(I left autoCorrect at the original places, because, while it's valid to define it, it won't have effect other than where it's already shown.)
Search (regex, spaces and new line at the end matter):
```
            "excludes": \{
              "type": "array",
              "items": \{
                "type": "string"
              }
            },?

```
Replace: nothing
,\s*"properties": \{\s*}
->
nothing
Inferred original detekt schema and example to be 1.14.1 based on date and contents.
@GerryFerdinandus
Copy link
Contributor

Thanks

@TWiStErRob TWiStErRob deleted the detekt-1.22.0 branch November 30, 2022 22:31
@TWiStErRob
Copy link
Contributor Author

Thanks for hosting! Amazing service.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants