bfinlay / laravel-excel-seeder

Seed your database with Laravel using Excel and CSV files
Other
38 stars 8 forks source link

install from composer gives you something weird? #20

Closed baradhili closed 11 months ago

baradhili commented 1 year ago

running composer require --dev bfinlay/laravel-excel-seeder gave me "bfinlay/laravel-excel-seeder": "^1.2"

Is it that I need to put the repo in for this one?

bfinlay commented 1 year ago

If you type composer require --dev bfinlay/laravel-excel-seeder:^3.0 does it give you any error messages?

baradhili commented 1 year ago

works fine, but without the version then it installs 1.2

bfinlay commented 11 months ago

I missed your reply previously. Did you resolve this?

I suspect this is an issue with version constraints for your composer packages.

When I run composer require --dev bfinlay/laravel-excel-seeder on a composer.json that does not have bfinlay/laravel-excel-seeder, it initially updates composer.json with "bfinlay/laravel-excel-seeder": "*", then it automatically runs composer update, then it updates composer.json with the latest release version, v3.3.

What is the output of this command? composer depends bfinlay/laravel-excel-seeder

bfinlay commented 11 months ago

Closed for now. If there is more information feel free to add a comment.

bfinlay commented 7 months ago

In connection with issue #22, I updated the readme.md to recommend using the -W switch with composer require.

I also added a test to github actions that creates a laravel app and runs composer require for each laravel and php configuration. Currently this test runs composer require without the --dev switch because that was the scenario in #22. The test also specifies the version as "dev-master" because I want to run the test against the tip.

You can look at the github actions tests here: https://github.com/bfinlay/laravel-excel-seeder/actions

image

bfinlay commented 7 months ago

I may add a test in the future that runs composer require and verifies that the latest version is installed.