cviebrock / eloquent-taggable

Easily add the ability to tag your Eloquent models in Laravel.
MIT License
559 stars 72 forks source link

Publish configuration laravel 5.6 #76

Closed loic-lopez closed 6 years ago

loic-lopez commented 6 years ago

Hello,

I use laravel 5.6 and your package version 3.3.1, and i ran to an issue with the publish command take a look bellow:

capture d ecran 2018-05-09 a 15 53 34

It does not publish the migration.

Thanks for reply 👍

cviebrock commented 6 years ago

I'm unable to reproduce this issue:

> laravel new l56
...

> cd l56
> composer require cviebrock/eloquent-taggable
Using version ^3.3 for cviebrock/eloquent-taggable
...

> php artisan migrate
Migration table created successfully.
Migrating: 0000_00_00_000000_create_taggable_table
Migrated:  0000_00_00_000000_create_taggable_table
Migrating: 2014_10_12_000000_create_users_table
Migrated:  2014_10_12_000000_create_users_table
Migrating: 2014_10_12_100000_create_password_resets_table
Migrated:  2014_10_12_100000_create_password_resets_table

Perhaps the confusion is from what I wrote in the documentation:

  1. Publish the configuration file and migrations:

In reality, only the configuration is published. The migrations are run out of the package directly (without needing to be published to /database/migrations). I'll update the docs.