Skip to content
Snippets Groups Projects

feat(var): make count of listed vars configurable

Merged Sebastian Gumprich requested to merge gitlab-community/cli:feat/var_ls_p into main
All threads resolved!

Description

  • feat(var): make count of listed vars configurable

Related Issues

Resolves #7749 (closed)

How has this been tested?

Before:

> glab var ls
Listing variables for the rndmh3ro/example project:

KEY            PROTECTED  MASKED  EXPANDED  SCOPE
foo            false      false   true      *
fooz           false      false   true      *
with_desc      false      false   true      *
a              false      false   true      *
a1             false      false   true      *
a2             false      false   true      *
a24            false      false   true      *
a214           false      false   true      *
a215           false      false   true      *
a21215         false      false   true      *
a212           false      false   true      *
a2121          false      false   true      *
a2121f         false      false   true      *
a2121f1        false      false   true      *
a2121f11       false      false   true      *
a2121f111      false      false   true      *
a2121f1111     false      false   true      *
a2121f11111    false      false   true      *
a2121f111111   false      false   true      *
a2121f1111111  false      false   true      *

After:

> ~/dev/glab/bin/glab var ls -P 30
Listing variables for the rndmh3ro/example project:

KEY               PROTECTED  MASKED  EXPANDED  SCOPE
foo               false      false   true      *
fooz              false      false   true      *
with_desc         false      false   true      *
a                 false      false   true      *
a1                false      false   true      *
a2                false      false   true      *
a24               false      false   true      *
a214              false      false   true      *
a215              false      false   true      *
a21215            false      false   true      *
a212              false      false   true      *
a2121             false      false   true      *
a2121f            false      false   true      *
a2121f1           false      false   true      *
a2121f11          false      false   true      *
a2121f111         false      false   true      *
a2121f1111        false      false   true      *
a2121f11111       false      false   true      *
a2121f111111      false      false   true      *
a2121f1111111     false      false   true      *
a2121f11111111    false      false   true      *
a2121f1111111121  false      false   true      *

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation
  • Chore (Related to CI or Packaging to platforms)
  • Test gap
Edited by 🤖 GitLab Bot 🤖

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Ezekiel Kigbo requested review from @timofurrer and removed review request for @ekigbo

    requested review from @timofurrer and removed review request for @ekigbo

  • Ezekiel Kigbo approved this merge request

    approved this merge request

  • Timo Furrer requested changes

    requested changes

  • Timo Furrer changed milestone to %17.10

    changed milestone to %17.10

  • added 1 commit

    • 75281c8e - chore: Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Sebastian Gumprich added 57 commits

    added 57 commits

    • 75281c8e...7ab36ce5 - 47 earlier commits
    • 7b26f065 - fix: accept input from stdin when stdin is TTY
    • a41f0cf9 - test: create with multiple files
    • f8e65d24 - feat: show help hint on error instead of full usage
    • 593ff709 - feat(ci status): return error when pipeline failed
    • 6c21c329 - feat(ssh-key): add usage type to output
    • 5ed9992b - fix(token-list): token list works outside a repository
    • ed25ca75 - fix(ci status): allow live to loop with NO_PROMPT
    • b3960473 - chore(deps): update module github.com/spf13/cobra to v1.9.1
    • 687c4982 - chore(deps): update module golang.org/x/crypto to v0.33.0
    • 50921df9 - add more tests

    Compare with previous version

  • Sebastian Gumprich added 104 commits

    added 104 commits

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • Timo Furrer resolved all threads

    resolved all threads

  • Timo Furrer approved this merge request

    approved this merge request

  • @rndmh3ro thanks for the contribution! I've approved and started the merge train 🚅 🎉

  • merged

  • Timo Furrer mentioned in commit ec23203f

    mentioned in commit ec23203f

  • Hi @rndmh3ro 👋

    🎉 See where you rank! Check your spot on the contributor leaderboard and unlock rewards.


    How was your code review experience with this merge request? Please tell us how we can continue to iterate and improve:

    1. React with a 👍 or a 👎 on this comment to describe your experience.
    2. Create a new comment starting with @gitlab-bot feedback below, and leave any additional feedback you have for us in the comment.

    As a benefit of being a GitLab Community Contributor, you have access to GitLab Duo, our AI-powered features. With Code Suggestions, Chat, Root Cause Analysis and more AI-powered features, GitLab Duo helps to boost your efficiency and effectiveness by reducing the time required to write and understand code and pipelines. Visit the GitLab Duo documentation to learn more about the benefits.

    Thanks for your help! ❤️

    This message was generated automatically. Improve it or delete it.

  • Please register or sign in to reply
    Loading