Skip to content
This repository was archived by the owner on Dec 19, 2023. It is now read-only.

Conversation

BlasiusSecundus
Copy link

Notable improvements:

  • upgrade Gradle Wrapper to version 6.2.1
  • add the Official Gradle Wrapper Validation Action
    (https://github.com/gradle/wrapper-validation-action)
  • use Spring dependency management
  • all dependency versions defined in gradle.properties
    (excluding those only used in examples)
  • update some dependency and plugin versions
  • update deprecated compile / runtime dependency configurations
    (replaced with implementation or api / runtimeOnly)
  • remove deprecated maven plugin
  • remove Lombok plugin (Lombok version is now managed by Spring)

Potential breaking changes:

  • the install task no longer exists; use publishToMavenLocal instead
  • Lombok-related tasks were removed

Note

This is quite a refactor and I hope that nothing is broken:) I have tested basic tasks like check, publishToMavenLocal etc. Also some tasks of the release plugin (e. g. unSnapshot, updateVersion).

I did not test (nor touched the configuration) of the Bintray plugin.

Please review / test it thoroughly to ensure that all necessary tasks still works as expected.

@oliemansm oliemansm added this to the 7.0.0 milestone Mar 3, 2020
@oliemansm
Copy link
Member

Thanks!

@oliemansm
Copy link
Member

@BlasiusSecundus Could you resolve the merge conflicts so I can go ahead and merge it?

Notable improvements:
- upgrade Gradle Wrapper to version 6.2.2
- add the Official Gradle Wrapper Validation Action
(https://github.com/gradle/wrapper-validation-action)
- use Spring dependency management
- all dependency versions defined in gradle.properties
(excluding those only used in examples)
- update some dependency and plugin versions
- update deprecated `compile` / `runtime` dependency configurations
(replaced with `implementation` or `api` / `runtimeOnly`)
- remove deprecated `maven` plugin
- remove Lombok plugin (Lombok version is now managed by Spring)

Potential breaking changes:

- the `install` task no longer exists; use `publishToMavenLocal` instead
- Lombok-related tasks were removed
@BlasiusSecundus BlasiusSecundus force-pushed the feature/build-script-refactor branch from d3b5ac0 to e7bb2ff Compare March 9, 2020 20:09
@BlasiusSecundus
Copy link
Author

BlasiusSecundus commented Mar 9, 2020

Conflicts resolved.

However, I could not make Voyager work, always got 404 (GraphiQL, Playground and Altair works file).

Also, I have not yet tested the reactive versions.

@BlasiusSecundus BlasiusSecundus changed the title chore: build script improvements, Gradle upgrade to 6.2.1 chore: build script improvements, Gradle upgrade to 6.2.2 Mar 9, 2020
@oliemansm oliemansm merged commit 0f948f8 into graphql-java-kickstart:master Mar 10, 2020
@oliemansm
Copy link
Member

Thanks!

@BlasiusSecundus BlasiusSecundus deleted the feature/build-script-refactor branch September 26, 2020 04:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants