Skip to content

Commit 0c83078

Browse files
Add the output option (#4015)
* Start of work on astroConfig.mode === 'server' * Add tests and more * adapter -> deploy in some places * Add fallback for `adapter` config * Update more tests * Update image tests * Fix clientAddress test * Updates based on PR review * Add a changeset * Update integrations tests + readme * Oops * Remove old option * Rename `mode` to `output` * Update Node adapter test * Update test * fred pass * fred pass * fred pass * fix test Co-authored-by: Fred K. Schott <fkschott@gmail.com>
1 parent d270cc7 commit 0c83078

3 files changed

Lines changed: 7 additions & 4 deletions

File tree

packages/adapters/node/README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@ import node from '@astrojs/node';
3737

3838
export default defineConfig({
3939
// ...
40+
output: 'server',
4041
adapter: node()
4142
})
4243
```

packages/adapters/node/src/index.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,12 @@ export default function createIntegration(): AstroIntegration {
1212
return {
1313
name: '@astrojs/node',
1414
hooks: {
15-
'astro:config:done': ({ setAdapter }) => {
15+
'astro:config:done': ({ setAdapter, config }) => {
1616
setAdapter(getAdapter());
17+
18+
if(config.output === 'static') {
19+
console.warn(`[@astrojs/Node] \`output: "server"\` is required to use this adapter.`);
20+
}
1721
},
1822
},
1923
};

packages/adapters/node/test/api-route.test.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,7 @@ describe('API routes', () => {
99
before(async () => {
1010
fixture = await loadFixture({
1111
root: './fixtures/api-route/',
12-
experimental: {
13-
ssr: true,
14-
},
12+
output: 'server',
1513
adapter: nodejs(),
1614
});
1715
await fixture.build();

0 commit comments

Comments
 (0)