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

Deprecate destinationDir option for v2.5.x #1154

Conversation

abelsromero
Copy link
Member

Kind of change

  • Bug fix
  • New non-breaking feature
  • New breaking feature
  • Documentation update
  • Build improvement

Description

What is the goal of this pull request?
Fix the option to set toDir internally.
Deprecate the option to warn users it is being removed in v3.0.x.

How does it achieve that?
Set toDir and add deprecation documentation.

Are there any alternative ways to implement this?
No

Are there any implications of this pull request? Anything a user must know?
No

Issue

If this PR fixes an open issue, please add a line of the form:

Fixes #853
Fixes #941

Release notes

Please add a corresponding entry to the file CHANGELOG.adoc

Fix option to set ToDir internally.
Deprecate destinationDir in Options and OptionsBuilder.
Add JavaDoc pointing users to use toDir.

Fixes asciidoctor#853
Fixes asciidoctor#941
@abelsromero abelsromero changed the base branch from main to v2.5.x March 30, 2023 21:13
@robertpanzer robertpanzer merged commit b76d453 into asciidoctor:v2.5.x Mar 31, 2023
@robertpanzer
Copy link
Member

Thank you!

@abelsromero abelsromero deleted the issue-853-deprecate-destinationDir-option-for-v2.5.x branch April 21, 2023 08:25
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.

Deprecation of Options destinationDir Destination directory ignored in Asciidoctorj
2 participants