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.

Sign up for GitHub

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

perf: reduce preload marker markup size #14550

Merged
merged 15 commits into from
Oct 10, 2023
Merged

perf: reduce preload marker markup size #14550

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
c41d3cd
perf: optimize preload markers
gajus Oct 7, 2023
c6aaaba
fix: allow runtime references
gajus Oct 7, 2023
fd6f4fd
style: apply prettier
gajus Oct 7, 2023
2201ddc
style: apply prettier
gajus Oct 7, 2023
1ffa2e2
fix: force build
gajus Oct 7, 2023
82efad9
refactor: move FileDep to the top
gajus Oct 9, 2023
926b2da
refactor: rename to __vite__mapDeps
gajus Oct 9, 2023
2a39b6d
Merge branch 'main' into gajus/optimize-preload-markers
gajus Oct 9, 2023
dc446ad
fix: force build
gajus Oct 9, 2023
435ced3
fix: update sourcemap
gajus Oct 9, 2023
1ca5017
refactor: use global __vite__fileDeps
gajus Oct 9, 2023
60b5969
Revert "refactor: use global __vite__fileDeps"
gajus Oct 9, 2023
0e9ee80
Update packages/vite/src/node/plugins/importAnalysisBuild.ts
gajus Oct 9, 2023
42a62b9
style: apply prettier
gajus Oct 9, 2023
b1f939e
refactor: avoid unnecessary ternary
gajus Oct 9, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
refactor: avoid unnecessary ternary
  • Loading branch information
@gajus
gajus committed Oct 9, 2023
commit b1f939e34c5f7997782df1e67a67913a57800df9
9 changes: 5 additions & 4 deletions packages/vite/src/node/plugins/importAnalysisBuild.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -579,10 +579,11 @@ export function buildImportAnalysisPlugin(config: ResolvedConfig): Plugin {
const cssDeps: number[] = []
const otherDeps: number[] = []
for (const dep of depsArray) {
;(getFileDep(dep).url.endsWith('.css')
? cssDeps
: otherDeps
).push(dep)
if (getFileDep(dep).url.endsWith('.css')) {
cssDeps.push(dep)
} else {
otherDeps.push(dep)
}
}
resolvedDeps = [
...resolveDependencies(
Expand Down