Fix Option values displaying as `Some("...")` in log output by Copilot · Pull Request #103 · f2calv/multi-arch-container-rust

Copilot · 2026-03-11T14:36:23Z

Original prompt

This section details on the original issue you should resolve

<issue_title>display Option better</issue_title>
<issue_description>Need to learn how to display Option better;

Git information; name 'Some("f2calv/multi-arch-container-rust")', branch 'Some("refs/heads/main")', commit 'Some("b7aacc0d228aff3c06f7d65b71fe91f0f22c4a7e")', tag 'Some("0.1.3")'</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes display Option<String> better #17

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

… format Co-authored-by: f2calv <16097639+f2calv@users.noreply.github.com>

Copilot

Pull request overview

Updates log formatting so Option values no longer render as Some("...") in Git/GitHub info logs (and in AppSettings’s Display output), improving readability of operational logs.

Changes:

Switched affected log statements from {:?} to {} and formatted Option<String> via .as_deref().unwrap_or("N/A").
Formatted Option<u64> / Option<u32> GitHub run fields into strings with an N/A fallback for missing values.
Updated Display impl for AppSettings to match the new Option formatting behavior.

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

You can also share your feedback on Copilot code review. Take the survey.

…/A literals Co-authored-by: f2calv <16097639+f2calv@users.noreply.github.com>

f2calv

The workflow is failing due to an external change in a GitHub Action, please fix, "The workflow is not valid. .github/workflows/ci.yml (Line: 47, Col: 11): Input image-registry is required, but not provided while calling. .github/workflows/ci.yml (Line: 47, Col: 11): Input image-repository is required, but not provided while calling."

f2calv

please also bump the latest cargo dependencies

…g changes Co-authored-by: f2calv <16097639+f2calv@users.noreply.github.com>

Copilot · 2026-03-20T14:28:43Z

The workflow is failing due to an external change in a GitHub Action, please fix, "The workflow is not valid. .github/workflows/ci.yml (Line: 47, Col: 11): Input image-registry is required, but not pr...

Fixed in commit 00f213b. The helm-chart-package.yml@v1 reusable workflow added new required inputs (image-registry, image-repository, chart-registry, chart-repository) and removed the old registry and chart-repository-prefix inputs. Updated the chart job accordingly:

image-registry: ghcr.io
image-repository: ${{ github.repository_owner }}/multi-arch-container-rust
chart-registry: ghcr.io
chart-registry-username: ${{ github.repository_owner }}
chart-repository: ${{ github.repository_owner }}/charts/multi-arch-container-rust
chart-path: charts/multi-arch-container-rust
Added secrets: inherit

Co-authored-by: f2calv <16097639+f2calv@users.noreply.github.com>

Copilot · 2026-03-20T14:30:57Z

please also bump the latest cargo dependencies

Done in commit 9c3cdd3. Ran cargo update — 20 packages updated to their latest compatible versions, including tokio (1.49 → 1.50), config (0.15.19 → 0.15.22), toml (0.9 → 1.0), libc (0.2.180 → 0.2.183), and others. Build verified successfully.

f2calv

@copilot increased scope, please review all the files in the root of this repo for updates/bugs and implement them, also bump the version of this repo with +semver:feature

rustup toolchain install stable-<cross-target> attempts to install and run binaries for the target platform on the build host, which fails with "may not be able to run on this system". Cross-compilation only requires rustup target add <target> (stdlib for the target) plus the system cross-linker from apt. Removed the three rustup toolchain install lines. +semver:feature Co-authored-by: f2calv <16097639+f2calv@users.noreply.github.com>

…t flag error The latest chart-testing image no longer accepts --set via --helm-extra-set-args in ct install, causing "unknown flag: --set". Removing the input lets the reusable workflow default to ct lint which validates the chart without attempting a cluster install. Co-authored-by: f2calv <16097639+f2calv@users.noreply.github.com>

… devcontainer - GitVersion.yml: add copilot branch pattern to prevent >63-char helm labels - deployment.yaml: add missing APP_ prefix to all env var names (runtime bug) - Cargo.toml: remove unused serde_json dependency - src/main.rs: fix misleading comment (INFO → DEBUG default log level) - Dockerfile: use absolute path in ENTRYPOINT - build.sh: fix shebang (sh → bash) to match bash-specific read -p usage - Chart.yaml: replace helm create placeholder appVersion 1.16.0 with 0.0.0 - .devcontainer: update to current mcr.microsoft.com/devcontainers/rust:1-bookworm - devcontainer.json: fix deprecated rust-analyzer.checkOnSave.command setting Co-authored-by: f2calv <16097639+f2calv@users.noreply.github.com> Agent-Logs-Url: https://github.com/f2calv/multi-arch-container-rust/sessions/2d38c791-ae0e-4533-98a2-0094712253fe

…eview feedback +semver:feature - GitVersion.yml: remove copilot branch pattern per reviewer request - Chart.yaml: restore helm-create default appVersion 1.16.0 per reviewer request Co-authored-by: f2calv <16097639+f2calv@users.noreply.github.com> Agent-Logs-Url: https://github.com/f2calv/multi-arch-container-rust/sessions/8bba02fe-79f8-47a6-8242-a74c0edd236e

f2calv

please make an empty commit with +semver:feature to bump this repos tag

Copilot · 2026-03-20T18:55:34Z

please make an empty commit with +semver:feature to bump this repos tag

Done — empty commit 0caa46e with +semver:feature.

Initial plan

33f3317

Copilot AI assigned Copilot and f2calv Mar 11, 2026

Copilot started work on behalf of f2calv March 11, 2026 14:36 View session

Fix Option display in log statements - use unwrap_or instead of Debug…

10aff00

… format Co-authored-by: f2calv <16097639+f2calv@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] Improve display of Option<String>~~ Fix Option values displaying as Some("...") in log output Mar 11, 2026

Copilot finished work on behalf of f2calv March 11, 2026 14:43

f2calv marked this pull request as ready for review March 12, 2026 19:41

f2calv requested a review from Copilot March 12, 2026 19:41

Copilot started reviewing on behalf of f2calv March 12, 2026 19:42 View session

Copilot AI reviewed Mar 12, 2026

View reviewed changes

Comment thread src/main.rs Outdated

Comment thread src/main.rs Outdated

Merge branch 'main' into copilot/improve-option-string-display

0de477b

Copilot started work on behalf of f2calv March 20, 2026 07:13 View session

Introduce DisplayOption wrapper to avoid allocations and duplicated N…

0920f00

…/A literals Co-authored-by: f2calv <16097639+f2calv@users.noreply.github.com>

Copilot AI requested a review from f2calv March 20, 2026 07:18

Copilot finished work on behalf of f2calv March 20, 2026 07:18

f2calv approved these changes Mar 20, 2026

View reviewed changes

Copilot started work on behalf of f2calv March 20, 2026 11:42 View session

Copilot AI requested a review from f2calv March 20, 2026 11:43

Copilot finished work on behalf of f2calv March 20, 2026 11:43

f2calv requested changes Mar 20, 2026

View reviewed changes

Copilot started work on behalf of f2calv March 20, 2026 14:24 View session

f2calv requested changes Mar 20, 2026

View reviewed changes

Fix CI: update chart job inputs for helm-chart-package.yml@v1 breakin…

00f213b

…g changes Co-authored-by: f2calv <16097639+f2calv@users.noreply.github.com>

Copilot AI requested a review from f2calv March 20, 2026 14:29

Copilot finished work on behalf of f2calv March 20, 2026 14:29

Copilot started work on behalf of f2calv March 20, 2026 14:29 View session

chore: bump cargo dependencies to latest compatible versions

9c3cdd3

Co-authored-by: f2calv <16097639+f2calv@users.noreply.github.com>

Copilot finished work on behalf of f2calv March 20, 2026 14:31

f2calv requested changes Mar 20, 2026

View reviewed changes

Copilot started work on behalf of f2calv March 20, 2026 15:07 View session

Copilot AI requested a review from f2calv March 20, 2026 15:12

Copilot finished work on behalf of f2calv March 20, 2026 15:12

Copilot started work on behalf of f2calv March 20, 2026 16:51 View session

Copilot finished work on behalf of f2calv March 20, 2026 16:54

f2calv added bug Something isn't working dependencies Pull requests that update a dependency file labels Mar 20, 2026

Copilot started work on behalf of f2calv March 20, 2026 18:38 View session

Copilot finished work on behalf of f2calv March 20, 2026 18:46

f2calv requested changes Mar 20, 2026

View reviewed changes

Comment thread charts/multi-arch-container-rust/Chart.yaml Outdated

Comment thread GitVersion.yml Outdated

Copilot started work on behalf of f2calv March 20, 2026 18:51 View session

Copilot AI requested a review from f2calv March 20, 2026 18:53

Copilot finished work on behalf of f2calv March 20, 2026 18:53

f2calv requested changes Mar 20, 2026

View reviewed changes

Copilot started work on behalf of f2calv March 20, 2026 18:54 View session

chore: bump version +semver:feature

0caa46e

Copilot AI requested a review from f2calv March 20, 2026 18:55

Copilot finished work on behalf of f2calv March 20, 2026 18:55

f2calv approved these changes Mar 20, 2026

View reviewed changes

f2calv merged commit 87b642b into main Mar 20, 2026
3 checks passed

f2calv deleted the copilot/improve-option-string-display branch March 20, 2026 18:57

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix Option values displaying as `Some("...")` in log output#103

Fix Option values displaying as `Some("...")` in log output#103
f2calv merged 11 commits into
mainfrom
copilot/improve-option-string-display

Copilot AI commented Mar 11, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

f2calv left a comment

Uh oh!

f2calv left a comment

Uh oh!

Copilot AI commented Mar 20, 2026

Uh oh!

Copilot AI commented Mar 20, 2026

Uh oh!

f2calv left a comment •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

f2calv left a comment

Uh oh!

Copilot AI commented Mar 20, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

Copilot AI commented Mar 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Comments on the Issue (you are @copilot in this section)

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

f2calv left a comment

Choose a reason for hiding this comment

Uh oh!

f2calv left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented Mar 20, 2026

Uh oh!

Copilot AI commented Mar 20, 2026

Uh oh!

f2calv left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

f2calv left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented Mar 20, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Copilot AI commented Mar 11, 2026 •

edited

Loading

f2calv left a comment •

edited

Loading