jenkins-infra / plugin-modernizer-tool

MIT License
10 stars 10 forks source link

Add modified files to the plugin repository #47

Open gounthar opened 5 months ago

gounthar commented 5 months ago

What feature do you want to see added?

I know we still have a long way to go before creating a pull request, but I wanted to share a thought with you.

I've used the tool on one of the plugins I maintain. Here's the command I entered:

java -jar plugin-modernizer-cli/target/jenkins-plugin-modernizer-999999-SNAPSHOT.jar --plugins android-emulator-plugin --recipes AddPluginsBom,AddCodeOwner

The output was:

Starting Plugin Modernizer
Plugins: [android-emulator-plugin]
Recipes: [AddPluginsBom, AddCodeOwner]
Invoking clean phase for plugin: android-emulator-plugin
Invoking rewrite plugin for plugin: android-emulator-plugin

My plugin didn't have a CODEOWNERS file, so the recipe created one for me (which is great!).

Looking ahead, when we integrate jgit or any other Git tool, should we consider, as a first step, adding new files to the Git repository? This could be an excellent initial step to familiarize ourselves with whichever Git tool we choose.

What are your thoughts on this approach?

Upstream changes

No response

Are you interested in contributing this feature?

No response

jonesbusy commented 5 months ago

I agree, let's keep it simple for the moment. The advantage of the AddCodeOwner is that it doesn't have any dependency. Plugin can be refreshed or not, it will always have the same behavior.

jonesbusy commented 3 months ago

First step on https://github.com/jenkinsci/plugin-modernizer-tool/pull/230

What we miss (not sure about the implementation yet)