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

Bump Gradle to 7.2 #170

Merged
merged 4 commits into from
Oct 28, 2021
Merged

Bump Gradle to 7.2 #170

merged 4 commits into from
Oct 28, 2021

Conversation

acogoluegnes
Copy link
Contributor

Fixes #169

resources {
from(sourceDir) {
include 'images/**'
include 'highlight/**/*'
Copy link
Contributor

Choose a reason for hiding this comment

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

don't think you have/need this since source highlighter is coderay

@@ -1,6 +1,6 @@
= Reactor RabbitMQ Reference Guide
Arnaud Cogoluègnes, Pawel Mackowski
// version is automatically set in doc.gradle, no need to change it here
// version is automatically set in asciidoc.gradle, no need to change it here
:appversion: 1.0.0.BUILD-SNAPSHOT
Copy link
Contributor

Choose a reason for hiding this comment

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

this is set in the configuration, so it can probably removed from here, just in case?

build.gradle Outdated
@@ -70,7 +68,7 @@ configure(allprojects) { project ->
maven { url 'https://repo.spring.io/snapshot' }
}

apply plugin: 'maven'
apply plugin: 'maven-publish'
Copy link
Contributor

Choose a reason for hiding this comment

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

this actually can be removed entirely, as maven-publish is applied in setup.gradle

After @simonbasle's remarks.

References #169
Because import org.gradle.util.VersionNumber is deprecated,
will be removed in Gradle 8.0, and Gradle does not replacement.

References #169
@acogoluegnes acogoluegnes merged commit 2bd7d1a into 1.4.x Oct 28, 2021
acogoluegnes added a commit that referenced this pull request Oct 29, 2021
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