Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

vendor以下での利用を可能に #1543

Closed
wants to merge 1 commit into from
Closed

Conversation

ttsuru
Copy link
Contributor

@ttsuru ttsuru commented Apr 17, 2016

refs #591, #864

EC-CUBE本体をcomposerディレクトリのおいた場合も動作するようにしました。
修正は以下になります。

  • vendor/ec-cube/ec-cube/ 以下に置くことが可能に
  • 上記に伴い、 path.yml の一部パラメータを廃止

本チケットのメリットは今後の複数バージョンでのプラグインテストが容易になることです。

以下の様なかたちで、 composer.json を記載するだけで、それぞれのバージョン環境が用意できます。
Pluginの自動テストに有効な対応となります。

{
    "require": {
        "ec-cube/ec-cube": "3.0.10"
    },
    "minimum-stability": "dev",
    "prefer-stable": true
}

@ttsuru ttsuru changed the title vendor以下での利用を可能に refs #591, #864 vendor以下での利用を可能に Apr 17, 2016
@ttsuru
Copy link
Contributor Author

ttsuru commented Jun 9, 2016

@Yangsin
こちらっていかがでしょうか。

@Yangsin
Copy link

Yangsin commented Jun 9, 2016

@ttsuru 反応おそくてすみません。
現時点ですぐすぐ採用とはいきませんが、1つの検討事項ではあります
プラグインの自動テスト自体は色々と検証をおこなっているのでまた情報展開します。

@ttsuru
Copy link
Contributor Author

ttsuru commented Jun 9, 2016

@Yangsin
このプルリクは通常のリリースには影響せず、開発やテストを容易にするないようですので、ぜひ早めに検討いただいたいです。
特に、バージョン違いのEC-CUBEでテストをすることを容易にします。

@Yangsin Yangsin added the enhancement 機能追加 label Jun 9, 2016
@Yangsin Yangsin added this to the 3.x.0 milestone Jun 9, 2016
@Yangsin
Copy link

Yangsin commented Jun 9, 2016

ご要望として受け止めさせていただきます。
今後のマイルストーンの提示と合わせて、後日回答させていただきます。

@ttsuru
Copy link
Contributor Author

ttsuru commented Aug 8, 2016

@Yangsin
ping

これって現時点で取り込みは難しいのでしょうか。
開発時のみに有効で、通常動作には影響がないプルリクだと思いますが・・・

とり込み可能であれば最新に合わせて修正します。

@Yangsin
Copy link

Yangsin commented Aug 8, 2016

こちら現時点では、技術的検証も不十分ながら、周辺環境を含めた影響範囲が読みきれておりませんので、マイルストーンを3.xとさせていただいております。
https://github.com/EC-CUBE/ec-cube/milestone/9

@Yangsin
Copy link

Yangsin commented Oct 13, 2016

こちら忘れてませんよというのと、今後のメジャーアップにむけて、もっとご意見募集という気持ちで discussion ラベルを付与しておきます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants