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

Fix DGPv2 migration helpers #3804

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

adam-enko
Copy link
Member

@adam-enko adam-enko commented Sep 17, 2024

Adjust the DGPv2 migration helpers to better support migration.

  • Create dokka${Format}${Subtype}Plugin (e.g. dokkaHtmlMultiModulePlugin) Configurations.
  • Now that DGP has HTML and Javadoc plugins, adjust the deprecated dokkaRuntime Configuration to redirect to the 'base' dokkaGeneratorRuntime Configuration instead of the HTML-specific dokkaHtmlGeneratorRuntime.

@adam-enko adam-enko added the runner: Gradle plugin An issue/PR related to Dokka's Gradle plugin label Sep 17, 2024
- Create `dokka${Format}${Subtype}Plugin` (e.g. `dokkaHtmlMultiModulePlugin`) Configurations.
- Now that DGP has HTML and Javadoc plugins, adjust the deprecated `dokkaRuntime` Configuration to redirect to the 'base' `dokkaGeneratorRuntime` Configuration instead of the HTML-specific `dokkaHtmlGeneratorRuntime`.
@adam-enko adam-enko force-pushed the adam/feat/KT-70336/fix-migration-helpers branch from 35eed2b to 5d0bfd8 Compare September 17, 2024 08:13
@adam-enko adam-enko marked this pull request as ready for review September 17, 2024 08:39
Copy link
Collaborator

@whyoleg whyoleg left a comment

Choose a reason for hiding this comment

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

LGTM

Note: KotlinDslAccessorsTest "dumps" are not really helpful here :(

@adam-enko
Copy link
Member Author

LGTM

Note: KotlinDslAccessorsTest "dumps" are not really helpful here :(

Well, their main purpose is to ensure that we don't accidentally break users' buildscripts by changing the generated accessors. This PR is intentionally modifying the accessors, so we expect changes. But yes, the the actual changes are difficult to discern.

@adam-enko adam-enko merged commit 9fc4da1 into master Sep 18, 2024
14 checks passed
@adam-enko adam-enko deleted the adam/feat/KT-70336/fix-migration-helpers branch September 18, 2024 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
runner: Gradle plugin An issue/PR related to Dokka's Gradle plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants