Open yeikel opened 1 year ago
@timtebeek We can create a separate recipe(should it be the same?) for Gradle
Probably best to create a separate recipe!
Probably best to create a separate recipe!
That's fair, I created https://github.com/openrewrite/rewrite-github-actions/issues/58
I am expecting a lot of code re-use here
You can assign both to me :)
Also, how do you think we should detect Windows vs Linux runners?
Few ideas:
I'd go with that second heuristic indeed; and likely not make any change if we can't be certain which is in use. That way we can support the most common use cases already, and people that need more can chime in again.
Also, how do you think we should detect Windows vs Linux runners?
Few ideas:
- Going off existing commands might only work if there are other steps we can use to predict
- In shared/public runners we can use something similar to this https://github.com/openrewrite/rewrite-github-actions/blob/main/src/main/java/org/openrewrite/github/PreferTemurinDistributions.java#L62
- For private runners, I think that we need user input
For an initial implementation, I would definitely like to support private runners(with user input) as well because this is the norm in Github Enterprise
I am open to implement private runners(GHE) and the public runners approach for the general github.com usage
@timtebeek Should we do this https://github.com/openrewrite/rewrite/issues/1565#issuecomment-1458562516
As part of this recipe just in case?
Can't hurt I suppose; as indeed, if that's not working the recipe will give bad results, loudly.
Hope you don't mind I unassigned you @yeikel I want to open this issue back up again to anyone to pick up. Feel free to open up a draft pull request if you'd like to contribute!
This recipe can be part of https://github.com/openrewrite/rewrite/issues/1565 and change all the maven references to the maven wrapper
Example:
Unix runners:
Windows Runners: