-
Notifications
You must be signed in to change notification settings - Fork 40
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
fix: manualChunksの指定が誤っていてビルド再現性がなかった #4150
fix: manualChunksの指定が誤っていてビルド再現性がなかった #4150
Conversation
Preview (prod) → https://4150-prod.traq-preview.trapti.tech/ |
// hljsは適当に二つに分割する | ||
const t = 'node_modules/highlight.js/lib/languages/' | ||
const firstLetter = id.slice(id.indexOf(t) + t.length)[0] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
t
がid
に含まれていないときid.indexOf(t)
は-1
なので、t.length - 1
番目の文字を見ていた
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4150 +/- ##
=======================================
Coverage 86.35% 86.35%
=======================================
Files 66 66
Lines 4719 4719
Branches 564 564
=======================================
Hits 4075 4075
Misses 638 638
Partials 6 6 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ありがとうございます!よさそうです!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
良さそうです。ビルド後hljs*が互いに依存してないことも確認できました
No description provided.