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

Add support for custom Ktlint RuleSets #2

Closed
wants to merge 12 commits into from

Conversation

Karn
Copy link
Owner

@Karn Karn commented Sep 30, 2022

A temporary branch for custom rules support while waiting for jeremymailen#279 to merge.

Usage:

buildscript {
  repositories {
    // ...
    maven { url "https://jitpack.io" }
  }
  
  dependencies {
    // ...
    classpath "io.karn:kotlinter-gradle:karn--enable-support-for-custom-rulesets-SNAPSHOT"
  }
}

If jitpack is down you can resolve via github:

buildscript {
  repositories {
    // ...
    maven {
      url "https://raw.githubusercontent.com/Karn/kotlinter-gradle/karn--enable-support-for-custom-rulesets-1/libs/"
      content {
        includeGroup("org.jmailen.gradle")
      }
    }
  }
  
  dependencies {
    // ...
    classpath "org.jmailen.gradle:kotlinter-gradle:3.12.0-local"
  }
}

Then add custom rulesets project

dependencies {
  // ...
  ktlintRuleset project(":build-ktlint-rules")
}

mateuszkwiecinski and others added 11 commits October 3, 2022 17:47
We don't use it, not sure why it is in there.
…n Java 19 (jeremymailen#306)

* Bump kotlin to 1.8.0

* Bump kotlinter to the latest version

* Bump com.gradle.plugin-publish to latest stable version

* Gradle 8.0 compatibility. Test on Gradle from 7.6, test on java 19
Co-authored-by: Mateusz Kwieciński <[email protected]>
- Fix issue jeremymailen#316 by upgrading kotlinter gradle's released version to ktlint 0.48.2.
- Remove configuration for `disabledRules` and `experimentalRules` to fully delegate this to .editorconfig
- Fix issue with resolution of rule enable / disable statements in .editorconfig
- Address issue jeremymailen#242 by providing readme example of excluding source files at the project tree level
- Address issue jeremymailen#314 by including sources in published artifacts
- Fix issue jeremymailen#313 by exporting `ReporterType`
- Address issue jeremymailen#315 by logging format log statements at level `WARNING` so they can be hidden
- Credit @mateuszkwiecinski as one of the developers of the plugin
@Karn Karn force-pushed the karn--enable-support-for-custom-rulesets branch from 7b91068 to b3045f7 Compare May 17, 2023 17:25
@Karn Karn force-pushed the karn--enable-support-for-custom-rulesets branch from b3045f7 to 0367e46 Compare May 17, 2023 17:39
@Karn Karn closed this May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants