.gitignore: Remove /vendor

  • Resolved vielhuber

    (@vielhuber)


    2 years, 10 months ago

    When you install this plugin on production, push everything to git and checkout locally, the required /vendor folder is missing.

    This results in the following error:

    Warning: require_once(/***/wp-content/plugins/advanced-cron-manager/vendor/autoload.php): failed to open stream: No such file or directory in /***/wp-content/plugins/advanced-cron-manager/advanced-cron-manager.php on line 20

Fatal error: require_once(): Failed opening required '/***/wp-content/plugins/advanced-cron-manager/vendor/autoload.php' (include_path='.:/usr/share/php') in /***/wp-content/plugins/advanced-cron-manager/advanced-cron-manager.php on line 20

    Please remove the line /vendor from .gitignore.

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author Kuba Mikita

    (@kubitomakita)

    Hi, vendor directory shouldn’t be under version control and it’s a bad practice to commit the 3rd party code to your own repo.

    I suggest installing plugins with Composer and wpackagist or even using Bedrock by Roots if you want the whole website to be under version control.

    Thread Starter vielhuber

    (@vielhuber)

    Sorry, I have a different opinion on that. Just run

    find . -type d -name 'vendor'

    in your installations and you will find dozens of folders which are not ignored and necessary that an environment runs.

    In any case, your strategy is unusual and leads to everyone being forced to use Composer, wpackagist or Bedrock (which is sometimes in use on our projects, but by no means always).

    • This reply was modified 2 years, 10 months ago by vielhuber.
    Plugin Author Kuba Mikita

    (@kubitomakita)

    Ohh I think the problem is somewhere else. You’re installing our plugin from www.ads-software.com, correct?

    I just noticed the package has some unnecessary files, including .gitignore which is interfering with your version control.

    I’ll fix the build and make sure it has only required files, wich should make you able to commit everything from our plugin folder.

    Thread Starter vielhuber

    (@vielhuber)

    > You’re installing our plugin from www.ads-software.com, correct?

    Correct.

    > I’ll fix the build and make sure it has only required files, wich should make you able to commit everything from our plugin folder.

    Thanks in advance.

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘.gitignore: Remove /vendor’ is closed to new replies.