Skip to content

Conversation

@Benbentwo
Copy link
Member

@Benbentwo Benbentwo commented Jun 25, 2025

CleanShot 2025-07-01 at 15 11 27

This pull request introduces several changes across different areas of the codebase, including dependency updates, documentation enhancements, and component refactoring. The key highlights are the downgrade of the Node.js version, the addition of a new documentation guide for self-hosted runners, and the complete refactoring of the EmbedFile component to improve modularity and styling.

Dependency Updates:

  • .nvmrc: Downgraded Node.js version from v21.7.3 to v20.12.0 to align with compatibility requirements.

Documentation Enhancements:

  • docs/layers/github-actions/runs-on.mdx: Added a comprehensive guide on setting up and managing self-hosted runners, including step-by-step instructions, configuration examples, and troubleshooting tips.

Component Refactoring:

  • src/components/EmbedFile/index.js: Removed the EmbedFile component, which dynamically loaded and displayed file content, to simplify the codebase and improve maintainability.
  • src/components/File/index.tsx: Introduced a new File component in TypeScript, encapsulating file display functionality with improved structure and modularity.
  • src/components/File/index.css: Added new styles for the File component, including enhancements for line numbering, tabs, and viewport design, with support for light and dark themes.

@Benbentwo Benbentwo requested a review from Copilot July 1, 2025 19:11
Copy link
Contributor

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 adds a new “Runs On” self-hosted runners guide and refines the File component’s styling and implementation.

  • Introduces a detailed MDX guide for setting up and using self-hosted GitHub Actions runners.
  • Adds CSS rules to support line numbering, tabs, and viewport styling in the File component.
  • Implements a basic React File component (TSX) and removes the deprecated EmbedFile loader.

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/components/File/index.tsx New React File component rendering a title tab and content viewport
src/components/File/index.css Styles for line numbers, tabs, and viewport in code display
src/components/EmbedFile/index.js Removed unused EmbedFile component
docs/layers/github-actions/runs-on.mdx Added “Runs On” guide with setup steps, examples, and troubleshooting
.nvmrc Downgraded Node.js version from v21.7.3 to v20.12.0
Comments suppressed due to low confidence (2)

.nvmrc:1

  • [nitpick] Ensure this Node.js version aligns with your package.json engines field and any CI configurations to avoid version mismatches.
v20.12.0

Co-authored-by: Erik Osterman (CEO @ Cloud Posse) <[email protected]>
@Benbentwo Benbentwo merged commit 061201e into master Jul 8, 2025
2 checks passed
@Benbentwo Benbentwo deleted the runs-on branch July 8, 2025 19:16
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.

4 participants