rename tasks rewriteWarn to rewriteDryRun and rewriteFix to rewriteRun #25
Conversation
| Task rewriteWarnAll = tasks.create("rewriteWarn", taskClosure(task -> { | ||
| Task rewriteDryRunAll = tasks.create("rewriteDryRun", taskClosure(task -> { |
There was a problem hiding this comment.
Let's keep the old "warn" name working for now, just in case someone uses it. Create another task named rewriteWarn that depends on rewriteDryRun. We'll leave the aliases in for a few releases
There was a problem hiding this comment.
Just to clarify: are we just aliasing the command rewriteWarn, or do we also need to alias rewriteWarn{SourceSet}, e.g. users running rewriteWarnTest, rewriteWarnMain, etc.?
There was a problem hiding this comment.
Probably just rewriteWarn. I think it unlikely that anyone will have bothered with the more granular per-sourceSet tasks
There was a problem hiding this comment.
Just for good measure I included the specific source sets, too.
aegershman
force-pushed
the
rewriteWarn-to-rewriteDryRun
branch
from
984a0b8
to
b363c0a
Compare
aegershman
force-pushed
the
rewriteWarn-to-rewriteDryRun
branch
from
b363c0a
to
0e52673
Compare
aegershman
changed the title
|
(going to close this PR outright and will open a separate one renaming both rewriteWarn and rewriteFix in the same PR and without aliasing) |
closes #20
should likely be followed by openrewrite/rewrite-maven-plugin#110