[help] trying to make it work on a scheduled composer update

[patrick-leb]

hi there!

You action works perfectly on my PR workflow, no problem there.
I'd like to commit a composer update on a schedule. here is my workflow:

'on':
  schedule:
    -
      cron: '24 6 * * 1,2,3,4,5'
  workflow_dispatch: null
jobs:
  composer_update_job:
    runs-on: ubuntu-latest
    name: 'composer update'
    steps:
    
      - name: Get current date
        id: date
        run: echo "::set-output name=date::$(date +'%Y-%m-%d')" 
      -
        name: Checkout
        uses: actions/checkout@v3
        with:
          fetch-depth: 0
      -
        name: 'Setup PHP'
        uses: shivammathur/setup-php@v2
        with:
          php-version: '8.0'
      -
        name: 'Run composer update'
        run: 'composer update --no-dev --no-progress'
        id: run_composer
        
      - name: Generate composer diff
        id: composer_diff # To reference the output in comment
        uses: IonBazan/composer-diff-action@v1
        with: 
          no-dev: true
          format: mdlist
      -
        name: 'Create Pull Request'
        uses: peter-evans/create-pull-request@v4
        id: cpr
        with:
          token: '${{ secrets.MERGE_TOKEN }}'
          commit-message: ${{ steps.composer_diff.outputs.composer_diff }}
          committer: 'Woogo Bot <[email protected]>'
          author: 'Woogo Bot <[email protected]>'
          branch: woogobot/composer-update
          delete-branch: true
          title: 'feat(plugins) composer update'
          labels: 'composer update'

The output of ${{ steps.composer_diff.outputs.composer_diff }} is empty there. However in the PR check, it will be correct.

What am I doing wrong here ?

Thanks!

[IonBazan]

Hi @patrick-leb, I'm glad you enjoy using this GitHub Action on your pipelines.
I think the problem is that base parameter defaults to ${{ github.event.pull_request.base.sha }} which is empty when the action is not triggered by a PR:

composer-diff-action/action.yml

Lines 8 to 11 in 5b081f9

	base:
	description: 'Base composer.lock file path or git ref '
	required: true
	default: ${{ github.event.pull_request.base.sha }}

When providing an empty string to base, the library concats composer.lock to it, causing a comparison between same file, producing an empty output. You can try setting it to HEAD in your workflow:

      - name: Generate composer diff
        id: composer_diff # To reference the output in comment
        uses: IonBazan/composer-diff-action@v1
        with: 
          base: HEAD
          no-dev: true
          format: mdlist

I hope that helps!

[patrick-leb]

That fixed it! Amazing!

May I suggest you add this use case to the docs? it will probably help someone else :)

Cheers!

[IonBazan]

Thanks for the suggestion - added it in the README.md since https://github.com/IonBazan/composer-diff-action/releases/tag/v1.0.6