Skip to content
This repository has been archived by the owner on Nov 22, 2024. It is now read-only.

[gradle] Remove dokka plugin #4995

Closed
wants to merge 1 commit into from
Closed

[gradle] Remove dokka plugin #4995

wants to merge 1 commit into from

Conversation

passy
Copy link
Member

@passy passy commented Aug 4, 2023

[gradle] Remove dokka plugin

Summary:
This is currently breaking our jetpack-compose build and blocks the entire release
as a consequence: #4970

There are all sorts of workarounds but they require changing the java target
which could have other downstream consequences: Kotlin/dokka#2956

We don't actually rely on these artefacts at all, so it's probably safe to just disable this.

Closes #4970

Test Plan:
./gradlew publishToMavenLocal

Summary:
This is currently breaking our jetpack-compose build and blocks the entire release
as a consequence: #4970

There are all sorts of workarounds but they require changing the java target
which could have other downstream consequences: Kotlin/dokka#2956

We don't actually rely on these artefacts at all, so it's probably safe to just disable this.

Closes #4970

Test Plan:
./gradlew publishToMavenLocal
@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Aug 4, 2023
@facebook-github-bot
Copy link
Contributor

@passy has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 4, 2023

Pull Request Test Coverage Report for Build 5760747359

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 51.369%

Totals Coverage Status
Change from base Build 5750082431: 0.0%
Covered Lines: 8338
Relevant Lines: 15062

💛 - Coveralls

@facebook-github-bot
Copy link
Contributor

@passy merged this pull request in 24cfd74.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed Do not delete this pull request or issue due to inactivity. Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GitHub Actions Workflow failed - Publish Android
2 participants