qcastel / github-actions-maven-release

Release your Java application and publish artifacts
129 stars 38 forks source link

Suggestion for doc update #45

Closed nbauma109 closed 2 years ago

nbauma109 commented 2 years ago

Thank you for this useful workflow.

I tried to follow the doc linearly from the beginning and I chose the access token which seemed like a quick win although it seems you recommend the ssh key. In fact it turned out to be more complicated:

To help users having a quick start: => direct them directly to ssh keys => JAVA_HOME is mandatory

Therefore I think the typical minimal setup looks like:

    - name: Release
      uses: qcastel/github-actions-maven-release@master
      env:
        JAVA_HOME: /usr/lib/jvm/java-17-openjdk/
      with:
       ssh-private-key: ${{ secrets.SSH_PRIVATE_KEY }}
qcastel commented 2 years ago

Hello @nbauma109,

Seems you had quite a journey to get your CI operational. Indeed always found the access token way quite complex for no reason, but in the past, I remember helping some guys getting it working. Although I agree, SSH seems a better approach for a first time so I will implement your suggestion. The fact I myself use SSH for all my repos is probably inducing that the SSH approach will always work better.

For the JAVA_HOME, didn't know that was mandatory to get it working so will add it too.

Thanks for your contribution to make this github action easier to use, don't hesitate to poke me if you think something else can be improved.

qcastel commented 2 years ago

Suggestion implemented here: https://github.com/qcastel/github-actions-maven-release#basic-setup

Thanks!