Skip to content

Fix typo and a couple minor log updates#13436

Merged
mitesch merged 1 commit intomainfrom
mitesch/typo
Dec 15, 2025
Merged

Fix typo and a couple minor log updates#13436
mitesch merged 1 commit intomainfrom
mitesch/typo

Conversation

@mitesch
Copy link
Contributor

@mitesch mitesch commented Dec 15, 2025

Description

Fixing a typo and took the opportunity to add some log changes

#13423

Related issue:

Check list

  • Related issue / work item is attached
  • Tests are written (if applicable)
  • Documentation is updated (if applicable)
  • Changes are tested and related VM images are successfully generated

Copilot AI review requested due to automatic review settings December 15, 2025 17:24
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 fixes a spelling error in comments and improves logging output in helper scripts. The changes focus on correcting "evironments" to "environment" and enhancing error handling with more informative log messages.

Key Changes:

  • Corrected spelling of "/etc/environment" in comments across helper scripts
  • Added error handling and logging to docker build command
  • Enhanced checksum verification success message to include hash values

Reviewed changes

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

File Description
images/ubuntu/scripts/helpers/etc-environment.sh Fixed spelling of "evironments" to "environment" in comment
images/ubuntu-slim/test.sh Added error handling and no-cache flag to docker build with explicit error logging
images/ubuntu-slim/scripts/helpers/install.sh Enhanced checksum verification success message to include actual hash values
images/ubuntu-slim/scripts/helpers/etc-environment.sh Fixed spelling of "evironments" to "environment" in comment
Comments suppressed due to low confidence (1)

images/ubuntu-slim/scripts/helpers/etc-environment.sh:1

  • Corrected spelling of 'evironments' to 'environment'.

@mitesch mitesch merged commit 7afeca2 into main Dec 15, 2025
5 checks passed
@mitesch mitesch deleted the mitesch/typo branch December 15, 2025 17:59
@mitesch mitesch mentioned this pull request Dec 15, 2025
4 tasks
dwydler pushed a commit to dwydler/runner-images-hetzner-cloud that referenced this pull request Dec 17, 2025
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