laravel-eloquent-join icon indicating copy to clipboard operation
laravel-eloquent-join copied to clipboard

Update to Laravel 9.

Open mikebronner opened this issue 3 years ago • 3 comments

mikebronner avatar Feb 22 '22 10:02 mikebronner

Tests run and pass locally. Not sure what the issue is with the GitHub workflow, as composer.json is in a valid format. Feel free to edit as necessary.

mikebronner avatar Feb 22 '22 10:02 mikebronner

Hi @fico7489, will this be investigated?

badasukerubin avatar Jun 02 '22 10:06 badasukerubin

Something new?

oksim14 avatar Jun 13 '22 14:06 oksim14

Any news with this PR? @fico7489

eliurkis avatar Nov 15 '22 03:11 eliurkis

Any news with this PR? @fico7489

HI @eliurkis, as a work-around I have been doing the following in my composer.json until this package gets updated:

{
    [...],
    repositories: [
        [...]
        {
            "type": "vcs",
            "url": "https://github.com/GeneaLabs/laravel-eloquent-join"
        }
        [...]
    ],
    require: {
        [...]
        "fico7489/laravel-eloquent-join": "dev-master@dev",
        [...]
    },
    [...]
}

This will install this PR.

mikebronner avatar Nov 15 '22 12:11 mikebronner

@mikebronner This update should fix the issue with the testing that you have.

image
  - name: Run find-and-replace to replace * with 0
    uses: mad9000/actions-find-and-replace-string@1
    id: laravel_version_cleaned
    with:
      source: ${{ matrix.laravel_version }}
      find: '*'
      replace: '0'

  - name: Install Laravel
    run: composer update --no-interaction illuminate/database:^${{ steps.laravel_version_cleaned.outputs.value }}

eliurkis avatar Nov 15 '22 20:11 eliurkis

@eliurkis very nice, thanks!

mikebronner avatar Nov 15 '22 22:11 mikebronner

@fico7489 Can you please merge the PR ?

thoss avatar Dec 05 '22 11:12 thoss

done

fico7489 avatar Dec 06 '22 07:12 fico7489