Yii2: Remove subtree split (yii2 / yii2-dev)

Created on 25 Apr 2018  路  6Comments  路  Source: yiisoft/yii2

What steps will reproduce the problem?

Have a project with dependency yiisoft/yii2.
Find a tiny bug, fix it and try to push it to your fork.

What is the expected result?

Easy creation of a PR.

What do you get instead?

This branch is 7469 commits ahead, 16992 commits behind yiisoft:master.

I need to use yii2-dev from the very beginning and switching within a project is hard (especially temporary). There should be only one repo for Yii2.

Change could be done in 2.1/3.0

ready for adoption

Most helpful comment

with .gitattributes we can exclude files when a zip file is created from the repo, so composer --prefer-source will install everything, --prefer-dist will only install non-dev files.

All 6 comments

As far as I understand yii team prefers to have all the framework (PHP files, documentation, tests and more) in single repository to be able to have single PR with everything.
But obviously users prefer only PHP files to be installed with composer.
Framework could be split into several repositories: one with source and other(s) with everything else.
But GitHub doesn't allow single PR to multiple repository.

I think this is a matter of file/directory-size in vendor.

156K    build
4.0K    code-of-conduct.md
4.0K    composer.json
84K composer.lock
24K contrib
24K cs
4.0K    Dockerfile
29M docs
6.2M    framework
4.0K    LICENSE.md
4.0K    package.json
4.0K    phpunit.xml.dist
8.0K    README.md
8.0K    ROADMAP.md
4.2M    tests

You'd get additional >30 MB of docs and tests.

with .gitattributes we can exclude files when a zip file is created from the repo, so composer --prefer-source will install everything, --prefer-dist will only install non-dev files.

Let's have .gitattributes

@schmunk42 can you handle it?

I think this is also fixed by having yiisoft/yii-core now.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

spiritdead picture spiritdead  路  67Comments

rosancoderian picture rosancoderian  路  46Comments

vercotux picture vercotux  路  47Comments

schmunk42 picture schmunk42  路  125Comments

Faryshta picture Faryshta  路  48Comments