Skip to content

Render explicit PowerShell Format-Table output from format metadata instead of fixed-width text parsing#3

Merged
eosfor merged 7 commits into
mainfrom
copilot/fix-html-table-rendering
May 17, 2026
Merged

Render explicit PowerShell Format-Table output from format metadata instead of fixed-width text parsing#3
eosfor merged 7 commits into
mainfrom
copilot/fix-html-table-rendering

Conversation

Copy link
Copy Markdown

Copilot AI commented May 17, 2026

  • Preserve intentionally blank table headers in metadata-driven Format-Table HTML rendering
  • Preserve Format-Table group sections in metadata rendering (e.g., -GroupBy)
  • Add/adjust PowerShell execution regression tests for header and group behavior
  • Run targeted PowerShell tests and confirm fixes
  • Run final validation (code review + CodeQL)

Copilot AI changed the title [WIP] Fix PowerShell Format-Table HTML parser issue for wide values Render explicit PowerShell Format-Table output from format metadata instead of fixed-width text parsing May 17, 2026
Copilot finished work on behalf of eosfor May 17, 2026 00:33
Copilot AI requested a review from eosfor May 17, 2026 00:33
@eosfor
Copy link
Copy Markdown
Owner

eosfor commented May 17, 2026

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c8b4426cd7

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/Verso.PowerShell/Kernel/RunspaceManager.cs Outdated
Comment thread src/Verso.PowerShell/Kernel/RunspaceManager.cs
Copilot finished work on behalf of eosfor May 17, 2026 01:00
@eosfor eosfor marked this pull request as ready for review May 17, 2026 01:19
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@eosfor
Copy link
Copy Markdown
Owner

eosfor commented May 17, 2026

@copilot can you review?

Copy link
Copy Markdown
Author

Copilot AI commented May 17, 2026

@copilot can you review?

Reviewed. Current PR changes look good from my side: Codex review threads are resolved and targeted PowerShell tests pass locally (dotnet test tests/Verso.PowerShell.Tests/Verso.PowerShell.Tests.csproj). Latest commit on this branch is 593759e.

Copilot finished work on behalf of eosfor May 17, 2026 01:22
@eosfor eosfor merged commit 5ea8ce4 into main May 17, 2026
1 check passed
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