Skip to content

feat: add --limit flag to list commands#57

Merged
piekstra merged 1 commit intomainfrom
piekstra/17-add-limit-flag
Jan 23, 2026
Merged

feat: add --limit flag to list commands#57
piekstra merged 1 commit intomainfrom
piekstra/17-add-limit-flag

Conversation

@piekstra
Copy link
Copy Markdown
Collaborator

@piekstra piekstra commented Jan 23, 2026

[#17]

Add --limit/-l flag to all list commands for shell scripting composability.

Changes

Added --limit flag to:

  • apps list
  • alerts policies list
  • dashboards list
  • users list
  • synthetics list
  • logs rules list

Notes on Issue #17

This PR partially addresses #17:

--limit flag (completed):

  • Added to all list commands that were missing it
  • deployments list and deployments search already had --limit

--force flag (already complete / N/A):

  • logs rules delete already has --force implemented
  • Delete commands for dashboards, deployments, and synthetics don't exist yet in the codebase

Test Plan

  • Build passes: make build
  • Tests pass: make test
  • Verify --limit flag appears in help: ./newrelic-cli apps list --help
  • Test limit truncation: ./newrelic-cli apps list --limit 1

@piekstra piekstra requested a review from rianjs January 23, 2026 02:53
Add --limit/-l flag to apps, alerts, dashboards, users, synthetics,
and logs rules list commands for shell scripting composability.

[#17]
@piekstra piekstra force-pushed the piekstra/17-add-limit-flag branch from 2299d33 to c6ecb43 Compare January 23, 2026 16:58
@piekstra piekstra merged commit c5eff17 into main Jan 23, 2026
2 checks passed
@piekstra piekstra deleted the piekstra/17-add-limit-flag branch January 23, 2026 18:02
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