Skip to content

Show markdown files as previews#52

Merged
kevinbackhouse merged 2 commits intoGitHubSecurityLab:mainfrom
kevinbackhouse:readme-preview
Feb 14, 2026
Merged

Show markdown files as previews#52
kevinbackhouse merged 2 commits intoGitHubSecurityLab:mainfrom
kevinbackhouse:readme-preview

Conversation

@kevinbackhouse
Copy link
Copy Markdown
Contributor

Open markdown files in preview mode, making them easier to read. I also removed the automatic open of README.md because it's done better by post-attach.sh.

Copilot AI review requested due to automatic review settings February 12, 2026 19:00
Copy link
Copy Markdown
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

Updates the devcontainer’s VS Code configuration to make Markdown files open in preview by default, and removes the Codespaces-specific openFiles behavior.

Changes:

  • Associate *.md with VS Code’s Markdown preview editor via workbench.editorAssociations.
  • Remove codespaces.openFiles that previously auto-opened README.md.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .devcontainer/devcontainer.json Outdated
Comment on lines 41 to 42
}
},
"codespaces": {
"openFiles": [
"README.md"
]
}
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

PR description says removing codespaces.openFiles is OK because post-attach.sh opens README.md, but post-attach.sh only runs code README.md when .env does not exist (first attach). After this change, README will no longer auto-open on subsequent attaches/sessions. Either update post-attach.sh to open README unconditionally (or under a more appropriate condition), or adjust the PR description/expectation.

Copilot uses AI. Check for mistakes.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings February 12, 2026 19:26
@kevinbackhouse kevinbackhouse marked this pull request as ready for review February 12, 2026 19:26
Copy link
Copy Markdown
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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kevinbackhouse kevinbackhouse merged commit 92cfaf2 into GitHubSecurityLab:main Feb 14, 2026
12 checks passed
@kevinbackhouse kevinbackhouse deleted the readme-preview branch February 14, 2026 16:49
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