Skip to content
This repository has been archived by the owner on Jun 1, 2023. It is now read-only.

[revisit sass] Feat/px2vw adopt nuxt.js with SASS math.div() working #6942

Closed
wants to merge 1 commit into from

Conversation

mcdmaster
Copy link
Contributor

@mcdmaster mcdmaster commented Nov 22, 2021

👏 解決する issue / Resolved Issues

📝 関連する issue / Related Issues

⛏ 変更内容 / Details of Changes

Every build time, our build tool looks like as if it deep-dives into the lowest level of enterire directories with our own or other open-source libraries / included items. In that case, we have been having some difficties like:

  • Firstly, in terms of "Changes" technically done, they are performed as workarounds and/or remediations to have our Nuxt.js instance (PROD assumed, naturally) keep itself up and running; they have been usually backed by testing excel accompanied with our Linting strategy
  • The bunches in tons ot error dumps, which SASS itself produces and throws them nudging the developer(s) to arise and start working on applying the change toward the use of SASS math.div() ffacility with advantage as early as he/she can;
  • Too much GBs of its log outputs are apparently threaten the developer's system resource, because they can easily terminate his/her good hacking activities with the network unexpedly occupied
  • Although it is figured out already that the most used and mass-volume application component Vuetify package has the greatest number of old SASS files, which produces tons of logs to exclaim that our system is 'deprecated' behind the latest SASS releases (here we can see that Vuetify is being outdated because of potential reluctancy of their changing mind)
  • At this time, SASS, sass-loader, Eslint etc are configured (as default) NOT TO be updated automatically; contrarily, this PR does NEVER cope with any kind of "downgrade" approach - just fix & go.
  • Hence I tried hard to suppress SASS message outputs on my end: I suppose this is almost completed but still requires sophistication and easy maintenability because it should not spend our hours to supplying Oxygen to old running Vuetify@2.

📸 スクリーンショット / Screenshots

@kaizumaki
Copy link
Collaborator

@mcdmaster すみません、こちらnetlifyのdeploy previewが動いていないので、ビルドスクリプトが根本的に正常に機能していないものと思われますが、いかがでしょう。

@mcdmaster
Copy link
Contributor Author

@kaizumaki ご心配とご迷惑を掛けております。

為念お伝えすると、「ビルドスクリプトが根本的に正常に機能していない」というほどのクリティカルなものではありません。
Lint 絡みで条件判定や、recommended テンプレをはじめとするルールに載らないコードを直していたら意外に時間が掛かってしまっています。
そのため、今の段階でもなお Ongoing 状態を招いているというのが正解です。

可能な限り早めの事態収拾を図りますので、もうしばらくお待ちいただけますか

@mcdmaster mcdmaster force-pushed the feat/px2vw branch 4 times, most recently from 8b0ace6 to 9ece14e Compare November 29, 2021 17:07
@mcdmaster mcdmaster force-pushed the feat/px2vw branch 12 times, most recently from 04d6967 to 032021d Compare December 25, 2021 18:13
@mcdmaster mcdmaster force-pushed the feat/px2vw branch 6 times, most recently from 4686b5c to 6be4b80 Compare December 31, 2021 01:48
@mcdmaster
Copy link
Contributor Author

こちら、なんだかどうにもラチが開かないので、いったんクローズしますね。
当初の目的は、単純に scss の文中で math.div() 関数を動かせるようにしたいだけだったんです。
しかしながら、聞き分けのない Vuetify チームともめてみたりとか、まあ、貴重な経験ではあったんですけど、正直疲れました。
Vue3 の適用を急ぐ方がむしろスムーズかもしれません

@mcdmaster mcdmaster closed this Feb 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants