Skip to content

Conversation

@rossiam
Copy link
Collaborator

@rossiam rossiam commented Jun 28, 2022

Update table output:
- switch to table package which handles international characters properly
- removed compact / expanded command line options
- removed compactTableOutput configuration option
- added group-rows and no-group-rows command line options
- added groupTableOutputRows configuration option
- (lib) completely isolated use of dependency to table-generator.ts

Checklist

  • I have read the CONTRIBUTING document
  • Any required documentation has been added
  • My code follows the code style of this project (npm run lint produces no warnings/errors)
  • I have added tests to cover my changes

@rossiam rossiam requested review from Sitlintac, bflorian and john-u June 28, 2022 17:34
@changeset-bot
Copy link

changeset-bot bot commented Jun 28, 2022

🦋 Changeset detected

Latest commit: 2d746d8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@smartthings/cli Patch
@smartthings/cli-lib Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

@john-u john-u left a comment

Choose a reason for hiding this comment

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

Awesome changes!

@rossiam rossiam merged commit 0849c4e into SmartThingsCommunity:master Jun 30, 2022
@rossiam rossiam deleted the new-table branch June 30, 2022 14:42
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