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

SCSSのビルドでstyle.css.mapファイルのコンフリクトが多発する #4332

Open
okazy opened this issue Sep 24, 2019 · 2 comments
Labels
css CSSの修正が必要なPR/Issue。スタイルガイドへの反映が必要 Status: discussion
Milestone

Comments

@okazy
Copy link
Contributor

okazy commented Sep 24, 2019

概要(Overview)

style.css.mapファイルもバージョン管理しているためSCSSを変更する複数のプルリクエストがあった場合にコンフリクトが必ず発生する。
style.css.mapファイルをバージョン管理に含めないと、cssのデバッグ時にビルドし直す必要があるため、style.css.mapファイルはバージョンに含めておきたい。

期待する内容(Expect) or 要望 (Requirement)

style.css.mapファイルをバージョン管理に含めるべきかどうかの意見があればコメントをお願いします。
また、何か良い解決策はないでしょうか?

再現手順(Procedure)

環境 (environment)

  • EC-CUBE: 4.0.x

関連情報 (Ref)

#4321 (comment)

@okazy okazy added Status: discussion css CSSの修正が必要なPR/Issue。スタイルガイドへの反映が必要 labels Sep 24, 2019
@okazy okazy added this to the 4.0.x milestone Sep 24, 2019
@ghost
Copy link

ghost commented Sep 24, 2019

style.css.map 管理に含めず、npm ci , npm run build:moc:sass まで、開発ドキュメントのインストールの要件に含まれていると開発する分には助かります。
npmスクリプトを明記することでminify化出来る選択肢があることを知ってもらえるのはメリットだと思います。

デメリットは沢山ありそうですね。

@ghost
Copy link

ghost commented Oct 3, 2019

触っていないファイル(app.css.mapbootstrap.css.map )が書き出されましたが、この影響ですよね。
PR出すときすごく悩んでしまいました。

@ghost ghost mentioned this issue Oct 10, 2019
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
css CSSの修正が必要なPR/Issue。スタイルガイドへの反映が必要 Status: discussion
Projects
None yet
Development

No branches or pull requests

1 participant