Skip to content

Conversation

Perdiga
Copy link
Collaborator

@Perdiga Perdiga commented Jul 9, 2025

No description provided.

@Copilot Copilot AI review requested due to automatic review settings July 9, 2025 20:43
@Perdiga Perdiga merged commit e79367a into main Jul 9, 2025
6 checks passed
Copy link

@Copilot Copilot AI left a comment

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 updates the Docusaurus baseUrl setting to use the root path (/) for both local development and GitHub Pages deployment.

  • Changed the comment to reflect that both environments now use /
  • Updated the baseUrl ternary to return / in all cases

// Use '/' for local development, '/codeql-wrapper/' for GitHub Pages
baseUrl: process.env.NODE_ENV === 'development' ? '/' : '/codeql-wrapper/',
// Use '/' for local development, '/' for GitHub Pages
baseUrl: process.env.NODE_ENV === 'development' ? '/' : '/',
Copy link
Preview

Copilot AI Jul 9, 2025

Choose a reason for hiding this comment

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

Since both development and production now use the same '/' path, the ternary operator is redundant. Consider simplifying to baseUrl: '/'.

Suggested change
baseUrl: process.env.NODE_ENV === 'development' ? '/' : '/',
baseUrl: '/',

Copilot uses AI. Check for mistakes.

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.

1 participant