Skip to content

refactor: replace custom table implementation with lipgloss table component#883

Merged
Soner (shyim) merged 2 commits intomainfrom
refactor/remove-table-component
Mar 2, 2026
Merged

refactor: replace custom table implementation with lipgloss table component#883
Soner (shyim) merged 2 commits intomainfrom
refactor/remove-table-component

Conversation

@shyim
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings March 2, 2026 08:42
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

This PR removes the custom internal/table rendering implementation and updates CLI commands to render tables using the charm.land/lipgloss/v2/table component directly.

Changes:

  • Deleted the custom table writer (internal/table/table.go).
  • Updated multiple cmd/* commands to build and render tables via lipgloss/v2/table.
  • Standardized table rendering in the touched commands to Border(lipgloss.NormalBorder()) + Headers(...) + Row(...).

Reviewed changes

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

Show a summary per file
File Description
internal/table/table.go Removes the bespoke table writer implementation.
cmd/project/project_extension_outdated.go Switches “outdated extensions” output to lipgloss/table.
cmd/project/project_extension_list.go Switches “extension list” output to lipgloss/table.
cmd/account/account_producer_extension_list.go Switches producer extension list output to lipgloss/table.
cmd/account/account_company_list.go Switches account company list output to lipgloss/table.

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

@shyim Soner (shyim) merged commit ccc6482 into main Mar 2, 2026
3 checks passed
@shyim Soner (shyim) deleted the refactor/remove-table-component branch March 2, 2026 08:50
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