Skip to content

MINOR: Gradle and Gradle plugins versions are extracted into 'gradle/dependencies.gradle' file#6648

Merged
ijuma merged 2 commits intoapache:trunkfrom
dejan2609:gradle-versions-extracted
May 2, 2019
Merged

MINOR: Gradle and Gradle plugins versions are extracted into 'gradle/dependencies.gradle' file#6648
ijuma merged 2 commits intoapache:trunkfrom
dejan2609:gradle-versions-extracted

Conversation

@dejan2609
Copy link
Copy Markdown
Contributor

@dejan2609 dejan2609 commented Apr 29, 2019

Committer Checklist (excluded from commit message)

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

@dejan2609
Copy link
Copy Markdown
Contributor Author

Prologue: #6436 (review)

@ijuma, @ewencp: please review and let me know do you prefer spotbugs_plugin over spotbugsPlugin (so I can change that)

Copy link
Copy Markdown
Member

@ijuma ijuma left a comment

Choose a reason for hiding this comment

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

Thanks for the PR. Just a couple of minor comments.

Comment thread gradle/dependencies.gradle Outdated
Comment thread gradle/dependencies.gradle Outdated
@dejan2609
Copy link
Copy Markdown
Contributor Author

dejan2609 commented Apr 30, 2019

Minor variable name changes (suggested by @ijuma) are applied.

@dejan2609
Copy link
Copy Markdown
Contributor Author

retest this please

Copy link
Copy Markdown
Member

@ijuma ijuma left a comment

Choose a reason for hiding this comment

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

LGTM, thanks.

@ijuma
Copy link
Copy Markdown
Member

ijuma commented May 2, 2019

The PR builder for Java 11 is currently failing due to an Apache Infra problem. There's no behaviour change in this PR so will go ahead and merge to trunk.

@ijuma ijuma merged commit 093a225 into apache:trunk May 2, 2019
@dejan2609 dejan2609 deleted the gradle-versions-extracted branch May 4, 2019 18:23
pengxiaolong pushed a commit to pengxiaolong/kafka that referenced this pull request Jun 14, 2019
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