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

Make 4.x the main development #662

Merged
merged 37 commits into from
Jan 5, 2024
Merged

Make 4.x the main development #662

merged 37 commits into from
Jan 5, 2024

Conversation

ysb33r
Copy link
Member

@ysb33r ysb33r commented Jan 2, 2023

This PR will eventually be merged to master when 4.0.0 final is released.

ysb33r and others added 30 commits August 6, 2022 15:49
- The main property is going to be replaced by mainClass in the JavaExec class.
- Dynamically attempt to continue calling the original property, but upon failure call the new property instead.
…ty if available, else revert to the old property

- This avoids any overhead in catching an exception if we're still running an older Gradle version.
- Upgraded Grolifant libray and adjusted use of ProjectOperations
  to avoid messages about configuration cache.
  Also updated package names where there were breaking changes.
- Updated NPM modules for exporting slides.
- Changed many 'compile' configurations to 'implementation' or
  'api'. Some are still there and needs fixing.
Support removal of main property in Gradle 8.0
Support latest worker API for Gradle 8.0 compatibility
Add `@IgnoreEmptyDirectories` where adequate
boostrack and others added 6 commits February 12, 2023 11:51
Asciidoctor task is relocatable when `gemPaths` is configured
This prepares 4.0.0 final release. It fully integrates with Grolifant 2.2
and adopts the newer JVM execution models too, whichs allows for better
integration with Gradle workers. This has been so successful that the
default execution model is no a worker with classpath isolation as
previously it always has been javaexec.

JRuby: jruby-gradle plugin has been replaced with jruby-simple-plugin.
  This eliminates a number of issues with the previous plugin in
  terms of functionality and compatibiliy. It also restores
  plugin compatibility with JDK8 whereas the older plugin requires
  JDK11.

GEMs: The issues with load errors for external GEMs has been
  resolved, but eliminating the use of GEM_PATH and rather
  compiling a custom GEM Jar on the fly and placing it on the
  classpath.

Disabled plugins: Some plugins were disabled for this
  release and will probably be fixed for future releases. Due to
  their limited usage, it was thought better to get a release out
  now and fix these later. The plugins are:

  - Leanpub
  - Slides
  - Slides export

Other:

- `asciidoctorEditorConfig` is now lazy-created
- Update to NodeJs plugin 2.0.1
- Remove most deprecated Grolifant 2.0 APIs

Known issues:

- Supplying an extension via a configuration rather than via
  `docExtensions` does not work.
- GEM resolver test on Windows is failing.
- Asciidotor.convertFile. Curretnly the Asciidoctor API does not
  allow for inspection of the Option object and the old Map methods
  are all deprecated.

Closes: #628, #626, #655, #658, #664, #671, #673, #684, i#692, #693
Remove most deprecated Grolifant 2.0 APIs
@ysb33r ysb33r marked this pull request as ready for review January 4, 2024 22:26
@ysb33r ysb33r added this to the 4.0.0 milestone Jan 4, 2024
@ysb33r ysb33r requested a review from aalmiray January 4, 2024 22:27
ysb33r referenced this pull request in apache/groovy Jan 4, 2024
Update asciidoctor-diagram.adoc
@aalmiray
Copy link
Member

aalmiray commented Jan 5, 2024

As far as I can tell LGTM. Local build failed but this PR ran successfully on GH

@ysb33r ysb33r merged commit 433c9dc into master Jan 5, 2024
6 checks passed
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.

7 participants