Make project.sync() a public API about gradle

[big-guy]

Sure, please make sure you take a look at our contributing guidelines.

[big-guy]

Thanks @hotchemi. I'll take a look.

Yes, those are both areas that we need to improve. You can leave those alone for this story. We have a general problem with the Copy-like tasks having under specified inputs like that.

[eriwen]

@big-guy We're booked for 3.3, so I think this should go in the next release after that unless it's very easy. Any thoughts? Is this easy enough we could ask for outside contribution?

[big-guy]

It's pretty easy.

We need to add the sync method with the signature WorkResult sync(Action<? super CopySpec> action); and documentation to subprojects/core/src/main/java/org/gradle/api/Project.java

We need to add the method to the DSL here: subprojects/docs/src/docs/dsl/org.gradle.api.Project.xml

Make sure we have decent test coverage, something like we do in CopyTaskIntegrationSpec and CopySpecIntegrationSpec in SyncTaskIntegrationTest or a new class.

[big-guy]

Note that we do not want a Closure version of this method.

[ajoberstar]

👍

[oconnelc]

I maybe able to take a look at it this weekend.

[hotchemi]

Can I grab the issue?

[hotchemi]

Addressed on #1137!
Btw, I found two NotYetImplemented annotations in integtests file. Would they be another story to enhance?

• gradle/subprojects/integ-test/src/integTest/groovy/org/gradle/integtests/SyncTaskIntegrationTest.groovy

  Line 173 in f001402

  @NotYetImplemented

• gradle/subprojects/integ-test/src/integTest/groovy/org/gradle/integtests/SyncTaskIntegrationTest.groovy

  Line 200 in f001402

  @NotYetImplemented

[hotchemi]

@big-guy Thank you for your response. Okay, so let me try to fix them when I can get a time to look into if it's okay.

[henryptung]

@big-guy Why no closure version?