Skip to content

feat: include a --no-wrap flag for table displays#3613

Open
heroku-johnny wants to merge 7 commits intomainfrom
table-no-wrap-flag
Open

feat: include a --no-wrap flag for table displays#3613
heroku-johnny wants to merge 7 commits intomainfrom
table-no-wrap-flag

Conversation

@heroku-johnny
Copy link
Copy Markdown
Contributor

Summary

Adds a --no-wrap flag to heroku domains so domain table output can be copied/pasted without wrapped cell values.
Default behavior is unchanged (overflow: 'wrap'), and --no-wrap switches table rendering to unbounded width for single-line values.

Type of Change

Breaking Changes (major semver update)

  • Add a ! after your change type to denote a change that breaks current behavior

Feature Additions (minor semver update)

  • feat: Introduces a new feature to the codebase

Patch Updates (patch semver update)

  • fix: Bug fix
  • deps: Dependency upgrade
  • revert: Revert a previous commit
  • chore: Change that does not affect production code
  • refactor: Refactoring existing code without changing behavior
  • test: Add/update/remove tests

Testing

Notes:
Validated behavior manually against a real app with long DNS target values.

Steps:

  1. npm run build
  2. node ./bin/run domains -a <app-name> (confirm default wrapped output remains)
  3. node ./bin/run domains -a <app-name> --no-wrap (confirm values remain on one line)
  4. node ./bin/run domains -a <app-name> | cat (reference output)
  5. npm run test:file -- test/unit/commands/domains/index.unit.test.ts

Screenshots (if applicable)

Related Issues

GitHub issue: #3606
GUS work item: W-21715803

Notes / Follow-up

Other commands currently using overflow: 'wrap' that may need the same --no-wrap option for consistency:

  • ps (src/commands/ps/index.ts)
  • pg:credentials (src/commands/pg/credentials.ts)
  • data:pg:credentials (src/commands/data/pg/credentials/index.ts)
  • addons (src/commands/addons/index.ts)

This Draft PR intentionally limits scope to domains only.

@heroku-johnny heroku-johnny changed the title feat: Include a --no-wrap flag for table displays feat: include a --no-wrap flag for table displays Mar 25, 2026
@heroku-johnny heroku-johnny changed the title feat: include a --no-wrap flag for table displays feat: include a --no-wrap flag for table displays Mar 25, 2026
@heroku-johnny heroku-johnny changed the title feat: include a --no-wrap flag for table displays feat: include a --no-wrap flag for table displays Mar 25, 2026
@heroku-johnny heroku-johnny marked this pull request as ready for review March 26, 2026 01:04
@heroku-johnny heroku-johnny requested a review from a team as a code owner March 26, 2026 01:04
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.

1 participant