Skip to content

Fix tablewriter API compatibility for v1.0.9 dependency upgrade#302

Merged
JAORMX merged 4 commits into
mainfrom
copilot/fix-25375b58-0aa5-4f3e-84ab-4117bd379150
Sep 12, 2025
Merged

Fix tablewriter API compatibility for v1.0.9 dependency upgrade#302
JAORMX merged 4 commits into
mainfrom
copilot/fix-25375b58-0aa5-4f3e-84ab-4117bd379150

Conversation

Copilot AI commented Sep 11, 2025

Copy link
Copy Markdown
Contributor
  • Fix tablewriter API compatibility for v1.0.9 dependency upgrade
  • Update table.SetHeader() to table.Header() calls in affected files
  • Add proper error handling for table.Append() and table.Render() methods
  • Ensure build passes successfully
  • Fix all linter issues related to the tablewriter upgrade

Changes Made

  • Updated API calls from SetHeader() to Header() in both cmd/actions/list.go and cmd/image/list.go
  • Added proper error handling for table.Append() calls with descriptive error messages
  • Added proper error handling for table.Render() calls with descriptive error messages
  • All linter issues resolved (errcheck violations for unchecked error returns)

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

@coveralls

coveralls commented Sep 11, 2025

Copy link
Copy Markdown

Coverage Status

coverage: 53.676% (-0.3%) from 53.941%
when pulling ad0172c on copilot/fix-25375b58-0aa5-4f3e-84ab-4117bd379150
into 7e5889b on main.

Copilot AI and others added 2 commits September 11, 2025 20:59
Co-authored-by: rdimitrov <16540482+rdimitrov@users.noreply.github.com>
Co-authored-by: rdimitrov <16540482+rdimitrov@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix the conflicts from bumping the dependency in @stacklok/frizbee/pull/261 Fix tablewriter API compatibility for v1.0.9 dependency upgrade Sep 11, 2025
Copilot AI requested a review from rdimitrov September 11, 2025 21:06
@rdimitrov

Copy link
Copy Markdown
Member

@copilot lets fix the linter issues now

Co-authored-by: rdimitrov <16540482+rdimitrov@users.noreply.github.com>

Copilot AI commented Sep 11, 2025

Copy link
Copy Markdown
Contributor Author

@copilot lets fix the linter issues now

Fixed all linter issues related to the tablewriter v1.0.9 upgrade. The new version requires error handling for table.Append() and table.Render() methods. Added proper error checking with descriptive error messages in both cmd/actions/list.go and cmd/image/list.go. All linter issues resolved with commit ad0172c.

@JAORMX JAORMX merged commit cf59988 into main Sep 12, 2025
16 checks passed
@JAORMX JAORMX deleted the copilot/fix-25375b58-0aa5-4f3e-84ab-4117bd379150 branch September 12, 2025 14:09
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.

5 participants