Skip to content

Cleanup hugo from devcontainer - #127

Merged
KittyChiu merged 1 commit into
mainfrom
feature/devcontainer-1
Jul 29, 2026
Merged

Cleanup hugo from devcontainer#127
KittyChiu merged 1 commit into
mainfrom
feature/devcontainer-1

Conversation

@KittyChiu

@KittyChiu KittyChiu commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

What changed

This pull request updates the development container configuration to use newer base images and features, and removes custom Go proxy and linter setup in favor of a simplified environment. The main changes focus on modernizing the development environment and removing custom Go-related tooling and configuration.

Development environment updates:

  • Updated the dev container base image to ubuntu24.04 and switched to the latest Node.js feature (node:2).
  • Added recommended VS Code extensions for TOML and Hugo, and removed the Go extension from the recommended list.

Go tooling and configuration cleanup:

  • Removed the Go proxy and Go linter features, as well as related environment variables and custom initialization scripts (on-create-command.sh was deleted). [1] [2] [3]
  • Removed custom Go linter flags from VS Code settings.
  • Removed Codespaces repository permissions configuration for Go linter and features.

Why

Type of change

  • Content Library article
  • Website code change
  • Documentation update
  • Other:

Contributors checklist

  • For Content Library changes: file name, directory, front matter, and writing style are appropriate
  • For website code changes: code standards, tests, dependencies, logging/debugging, and LICENSE impact were considered
  • For documentation updates: content is clear, accurate, consistent, and proofread
  • Related issue, discussion, or Content Request is linked, if applicable

Notes for reviewers

Copilot AI review requested due to automatic review settings July 29, 2026 02:58
@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Modernizes and simplifies the Hugo development container.

Changes:

  • Updates Ubuntu and Node.js container components.
  • Removes custom Go proxy and lint configuration.
  • Adds Hugo/TOML extensions and ignores generated lockfiles.
Show a summary per file
File Description
.gitignore Ignores the dev container lockfile.
.devcontainer/on-create-command.sh Removes obsolete Go initialization.
.devcontainer/devcontainer.json Modernizes and simplifies container configuration.

Review details

Tip

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

  • Files reviewed: 2/3 changed files
  • Comments generated: 1
  • Review effort level: Medium

Comment thread .devcontainer/devcontainer.json
@KittyChiu
KittyChiu merged commit 26610c8 into main Jul 29, 2026
12 checks passed
@KittyChiu
KittyChiu deleted the feature/devcontainer-1 branch July 29, 2026 03:01
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.

2 participants