Skip to content

Commit

Permalink
include artifacts in all builds
Browse files Browse the repository at this point in the history
  • Loading branch information
AlecAivazis committed Oct 16, 2023
1 parent 270edce commit ef1a1bb
Showing 1 changed file with 1 addition and 8 deletions.
9 changes: 1 addition & 8 deletions packages/houdini-react/src/plugin/vite.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ let devServer: boolean = false
export default {
// we want to set up some vite aliases by default
async config(config, env) {
manifest = await load_manifest({ config, includeArtifacts: env.mode === 'production' })
manifest = await load_manifest({ config, includeArtifacts: env.command === 'build' })
setManifest(manifest)

// secondary builds have their own rollup config
Expand All @@ -43,12 +43,6 @@ export default {
},
}

console.log({
isSecondaryBuild: isSecondaryBuild(),
env: process.env.HOUDINI_SECONDARY_BUILD,
artifacts: manifest.artifacts,
})

// build up the list of entries that we need vite to bundle
if (!isSecondaryBuild() || process.env.HOUDINI_SECONDARY_BUILD === 'ssr') {
if (env.command === 'build') {
Expand Down Expand Up @@ -77,7 +71,6 @@ export default {
] = `virtual:houdini/pages/${page.id}@${page.queries}.jsx`
}

console.log(manifest.artifacts)
// every artifact asset needs to be bundled individually
for (const artifact of manifest.artifacts) {
conf.build!.rollupOptions!.input[
Expand Down

0 comments on commit ef1a1bb

Please sign in to comment.