Skip to content

Conversation

@jake-mahon-netwrix
Copy link
Contributor

Summary

  • Removes DOCS_PRODUCT=kb from default build script
  • Adds build-kb script for KB-only builds when needed
  • Default build now builds all products with original 16GB memory limit
  • Keeps build configuration simple and consistent

Background

The previous change made the default build KB-only to test our documentation fixes. Now that the fixes are working, we need the production build to include all products while keeping the KB-only option available for development/testing.

Changes

  • npm run build - Full-site build with 16GB memory (original default behavior)
  • npm run build-kb - KB-only build with 16GB memory for testing/development

Test plan

  • Verified build scripts are correctly configured
  • Will test full site build in CI/CD

🤖 Generated with Claude Code

Jake Mahon and others added 3 commits September 16, 2025 07:52
- Remove DOCS_PRODUCT=kb from default build script
- Add build-kb script for KB-only builds when needed
- Default build now builds all products with original 16GB memory limit
- Keep build configuration simple and consistent

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Change onBrokenLinks from 'warn' to 'throw'
- Change onBrokenMarkdownLinks from 'warn' to 'throw'
- Change onBrokenAnchors from 'warn' to 'throw'
- Ensures builds fail on any broken links for quality control

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Remove build-kb script option
- Only keep default build script for full-site builds
- Simplifies build configuration and removes unnecessary options

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@james-s-anderson james-s-anderson merged commit c2c42e5 into dev Sep 16, 2025
3 checks passed
@james-s-anderson james-s-anderson deleted the change-build-to-full-site branch September 16, 2025 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants