Skip to content

MINOR: Add spotlessScalaCheck dependency to streams-scala test task#5479

Merged
guozhangwang merged 1 commit intoapache:trunkfrom
omkreddy:spotless
Aug 13, 2018
Merged

MINOR: Add spotlessScalaCheck dependency to streams-scala test task#5479
guozhangwang merged 1 commit intoapache:trunkfrom
omkreddy:spotless

Conversation

@omkreddy
Copy link
Contributor

@omkreddy omkreddy commented Aug 8, 2018

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

@omkreddy
Copy link
Contributor Author

omkreddy commented Aug 8, 2018

@guozhangwang added spotlessScalaCheck dependency to streams-scala test task, so that developers will get format errors while executing streams scala tests.

@omkreddy
Copy link
Contributor Author

@guozhangwang Pls take a look at this minor PR.

Copy link
Contributor

@guozhangwang guozhangwang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@guozhangwang guozhangwang merged commit 68fb293 into apache:trunk Aug 13, 2018
@omkreddy omkreddy deleted the spotless branch August 20, 2018 09:29
pengxiaolong pushed a commit to pengxiaolong/kafka that referenced this pull request Jun 14, 2019
ijuma pushed a commit that referenced this pull request Feb 24, 2023
Details:
 * gradle upgrade: 7.6 -> 8.0.1
 * spotbugs plugin upgrade: 5.0.9 -> 5.0.13
 * tweaked the mechanics for `-release`/`-source`/`-target` to workaround idiosyncrasies in Gradle 8.0.1 and newer Scala 2.13 versions.
 * streams-scala `test` task no longer triggers the `spotless` task since a newer version is required for Gradle 8 support, but the newer version requires Java 11.
Note: relates to #5479

Gradle upgrade highlights:
* "Scala Incremental Compilation for Multi-Module projects broken in 7.x": gradle/gradle#20101
* "Incremental compilation of java modules is broken with Gradle 7.6": gradle/gradle#23067

Full release notes: https://docs.gradle.org/8.0/release-notes.html

Reviewers: Ismael Juma <[email protected]>
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.

2 participants