Skip to content

Commit

Permalink
Upgrade publish plugin + Dokka
Browse files Browse the repository at this point in the history
Summary: Necessary due to Kotlin and Gradle upgrades. This upgrade caused a few doc inconsistencies to become errors so I fixed those as well. The `androidJavadoc` task also is no longer available in `afterEvaluate` so I moved the BUCK file exclusion to a more generic `whenTaskAdded` lambda

Differential Revision: D33961680

fbshipit-source-id: 4637af21aef7392b80bb06766bd648cdbd07d154
  • Loading branch information
xiphirx authored and facebook-github-bot committed Feb 3, 2022
1 parent a69a7ac commit f088bf8
Show file tree
Hide file tree
Showing 6 changed files with 29 additions and 13 deletions.
9 changes: 7 additions & 2 deletions core/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -40,8 +40,13 @@ android {

afterEvaluate {
generateReleaseBuildConfig.enabled = false
tasks.getByName("androidJavadocs") {
exclude '**/BUCK'
}

tasks.whenTaskAdded {
if (it.name == "androidJavadoc") {
it.configure {
exclude '**/BUCK'
}
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -65,9 +65,10 @@ public interface RecordBuilder {
* Set the maximum number of pixels this screenshot should produce. Producing any number higher
* will throw an exception.
*
* @param maxPixels Maximum number of pixels this screenshot should produce. <= 0 for no limit.
* @param maxPixels Maximum number of pixels this screenshot should produce. Specify zero or a
* negative number for no limit.
*/
public RecordBuilder setMaxPixels(long maxPixels);
RecordBuilder setMaxPixels(long maxPixels);

/** Finish the recording. */
void record();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,8 @@

/**
* This class provides utility methods for determining certain accessibility properties of {@link
* View}s and {@link AccessibilityNodeInfoCompat}s. It is porting some of the checks from {@link
* com.googlecode.eyesfree.utils.AccessibilityNodeInfoUtils}, but has stripped many features which
* View}s and {@link AccessibilityNodeInfoCompat}s. It is porting some of the checks from
* com.googlecode.eyesfree.utils.AccessibilityNodeInfoUtils, but has stripped many features which
* are unnecessary here.
*/
public class AccessibilityUtil {
Expand Down Expand Up @@ -127,7 +127,7 @@ public static AccessibilityRole getRole(View view) {
* Gets the role from a given {@link AccessibilityNodeInfoCompat}. If no role is defined it will
* return AccessibilityRole.NONE, which has a value of null.
*
* @param node The node to check.
* @param nodeInfo The node to check.
* @return {@code AccessibilityRole} the defined role.
*/
public static AccessibilityRole getRole(AccessibilityNodeInfoCompat nodeInfo) {
Expand Down
9 changes: 7 additions & 2 deletions layout-hierarchy-common/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -35,8 +35,13 @@ android {

afterEvaluate {
generateReleaseBuildConfig.enabled = false
tasks.getByName("androidJavadocs") {
exclude '**/BUCK'
}

tasks.whenTaskAdded {
if (it.name == "androidJavadoc") {
it.configure {
exclude '**/BUCK'
}
}
}

Expand Down
9 changes: 7 additions & 2 deletions layout-hierarchy-litho/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -35,8 +35,13 @@ android {

afterEvaluate {
generateReleaseBuildConfig.enabled = false
tasks.getByName("androidJavadocs") {
exclude '**/BUCK'
}

tasks.whenTaskAdded {
if (it.name == "androidJavadoc") {
it.configure {
exclude '**/BUCK'
}
}
}

Expand Down
4 changes: 2 additions & 2 deletions versions.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,8 @@ ext {
screenshot : "com.facebook.testing.screenshot:plugin:$screenshotTestVersion",
kotlin : "org.jetbrains.kotlin:kotlin-gradle-plugin:${versions.kotlin}",
versions : 'com.github.ben-manes:gradle-versions-plugin:0.20.0',
publish : 'com.vanniktech:gradle-maven-publish-plugin:0.14.2',
dokka : 'org.jetbrains.dokka:dokka-gradle-plugin:1.4.30'
publish : 'com.vanniktech:gradle-maven-publish-plugin:0.18.0',
dokka : 'org.jetbrains.dokka:dokka-gradle-plugin:1.6.10'
]

deps = [
Expand Down

0 comments on commit f088bf8

Please sign in to comment.