Skip to content

Commit 044913a

Browse files
committed
improvement(ci): preserve builds across developer documentation edits
1 parent 0335d6f commit 044913a

4 files changed

Lines changed: 8 additions & 6 deletions

File tree

‎.github/workflows/checks.yml‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -950,7 +950,6 @@ jobs:
950950
# lint's type-check and the `build` job's own `next build` (generated route types too)
951951
# already enforce.
952952
NEXT_BUILD_SKIP_TYPE_CHECK: 'true'
953-
NEXT_BUILD_DISABLE_SOURCE_MAPS: 'true'
954953
run: bunx turbo run build --filter=@sim/app
955954

956955
# Everything `next start` reads, including the spreadsheet parse worker it spawns (named, so a

‎apps/sim/lib/core/config/env.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -336,7 +336,6 @@ export const env = createEnv({
336336
// Infrastructure & Deployment
337337
NEXT_RUNTIME: z.string().optional(), // Next.js runtime environment
338338
DOCKER_BUILD: z.boolean().optional(), // Flag indicating Docker build environment
339-
NEXT_BUILD_DISABLE_SOURCE_MAPS: z.boolean().optional(), // Omits source-map processing from builds that do not serve maps
340339
NEXT_BUILD_SKIP_TYPE_CHECK: z.boolean().optional(), // Skips `next build`'s TypeScript pass where another CI job type-checks
341340

342341
// Background Jobs & Scheduling

‎apps/sim/next.config.ts‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ const nextConfig: NextConfig = {
3131
// Safe here since this repo's source is already fully public on GitHub -
3232
// no additional exposure versus Next's default (disabled to avoid leaking
3333
// source on the client).
34-
productionBrowserSourceMaps: !isTruthy(env.NEXT_BUILD_DISABLE_SOURCE_MAPS),
34+
productionBrowserSourceMaps: true,
3535
turbopack: {
3636
root: path.join(import.meta.dirname, '../..'),
3737
},
@@ -169,7 +169,6 @@ const nextConfig: NextConfig = {
169169
],
170170
},
171171
experimental: {
172-
turbopackInputSourceMaps: !isTruthy(env.NEXT_BUILD_DISABLE_SOURCE_MAPS),
173172
/**
174173
* Turbopack's dev filesystem cache stays ON (this is also the Next default
175174
* since v16.1). It is what makes a dev-server restart cheap: without it every

‎apps/sim/turbo.json‎

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,7 @@
165165
]
166166
},
167167
"build": {
168-
"env": ["NEXT_BUILD_DISABLE_SOURCE_MAPS", "NEXT_BUILD_SKIP_TYPE_CHECK"],
168+
"env": ["NEXT_BUILD_SKIP_TYPE_CHECK"],
169169
"inputs": [
170170
"$TURBO_DEFAULT$",
171171
".env*",
@@ -177,7 +177,12 @@
177177
"!**/CLAUDE.md",
178178
"!scripts/test-*-e2e.ts",
179179
"!vitest.*",
180-
"!test-results/**"
180+
"!test-results/**",
181+
"!ee/README.md",
182+
"!ee/access-requests/README.md",
183+
"!ee/scim/TESTING.md",
184+
"!lib/projects/README.md",
185+
"!tools/pitchbook/__fixtures__/README.md"
181186
]
182187
}
183188
}

0 commit comments

Comments
 (0)