Skip to content

SH-8243#333

Merged
timurscribe merged 3 commits into
masterfrom
SH-8243
Sep 9, 2025
Merged

SH-8243#333
timurscribe merged 3 commits into
masterfrom
SH-8243

Conversation

@timurscribe

Copy link
Copy Markdown
Contributor

No description provided.

@netlify

netlify Bot commented Sep 9, 2025

Copy link
Copy Markdown

Deploy Preview for scribe-security ready!

Name Link
🔨 Latest commit 7f7e601
🔍 Latest deploy log https://app.netlify.com/projects/scribe-security/deploys/68c00a986e0c930008f1d4ce
😎 Deploy Preview https://deploy-preview-333--scribe-security.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@timurscribe
timurscribe requested a review from cppvik September 9, 2025 11:09
@cppvik
cppvik requested a review from Copilot September 9, 2025 11:12

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR refactors the Docusaurus configuration to improve branch detection reliability and adds an environment variable to force production behavior. The changes include safer error handling for Git operations and code cleanup.

  • Enhanced branch detection with proper error handling and fallback to environment variables
  • Added FORCE_PROD environment variable to override branch-based behavior for production-like builds
  • Minor code formatting cleanup including trailing whitespace removal
Comments suppressed due to low confidence (1)

docusaurus.config.js:1

  • There's an extra closing bracket here that creates invalid syntax. The include array is being closed twice - once at line 201 and again at line 202. Remove this duplicate closing bracket.
// @ts-check

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread docusaurus.config.js
@timurscribe
timurscribe merged commit 18fc77b into master Sep 9, 2025
4 checks passed
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