Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(cli): support "-l" for "gas ls" to show total number of resources #864

Merged
merged 1 commit into from
Jul 29, 2021
Merged

feat(cli): support "-l" for "gas ls" to show total number of resources #864

merged 1 commit into from
Jul 29, 2021

Conversation

marshallmallows
Copy link
Collaborator

No description provided.

@marshallmallows marshallmallows requested a review from Lee-000 as a code owner July 28, 2021 08:16
@coveralls
Copy link

coveralls commented Jul 28, 2021 •
edited
Loading

Pull Request Test Coverage Report for Build 1077522898

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 83.044%
Totals Coverage Status
Change from base Build 1077466951: 0.0%
Covered Lines: 5265
Relevant Lines: 6340
💛 - Coveralls

Lee-000
Lee-000 requested changes Jul 28, 2021
View reviewed changes
tensorbay/cli/cli.py Show resolved Hide resolved
tensorbay/cli/ls.py Outdated Show resolved Hide resolved
tensorbay/cli/ls.py Outdated Show resolved Hide resolved
@marshallmallows marshallmallows force-pushed the T17056_add_l_for_gas_ls branch from 045b71f to 24ecd1f Compare July 28, 2021 09:44
@marshallmallows marshallmallows requested a review from Lee-000 July 28, 2021 09:45
Lee-000
Lee-000 reviewed Jul 28, 2021
View reviewed changes
tensorbay/cli/ls.py Outdated Show resolved Hide resolved
linjiX
linjiX requested changes Jul 29, 2021
View reviewed changes
tensorbay/cli/cli.py Outdated Show resolved Hide resolved
Lee-000
Lee-000 requested changes Jul 29, 2021
View reviewed changes
tensorbay/cli/cli.py Outdated Show resolved Hide resolved
@marshallmallows marshallmallows force-pushed the T17056_add_l_for_gas_ls branch from 24ecd1f to 0bb7151 Compare July 29, 2021 03:47
Lee-000
Lee-000 requested changes Jul 29, 2021
View reviewed changes
tensorbay/cli/cli.py Outdated Show resolved Hide resolved
@marshallmallows marshallmallows force-pushed the T17056_add_l_for_gas_ls branch from 0bb7151 to d07017d Compare July 29, 2021 04:29
@marshallmallows marshallmallows requested a review from Lee-000 July 29, 2021 04:35
@marshallmallows marshallmallows force-pushed the T17056_add_l_for_gas_ls branch from d07017d to d442ab5 Compare July 29, 2021 05:08
@marshallmallows marshallmallows changed the title feat(cli): support "-l" for "gas ls" to show total number of files feat(cli): support "-l" for "gas ls" to show total number of resources Jul 29, 2021
marshallmallows added a commit to marshallmallows/tensorbay-python-sdk that referenced this pull request Jul 29, 2021
@marshallmallows
feat(cli): support "-l" for "gas ls" to show total number of resources
4d9c510 
PR Closed: Graviti-AI#864
@marshallmallows marshallmallows force-pushed the T17056_add_l_for_gas_ls branch from d442ab5 to 4d9c510 Compare July 29, 2021 05:38
@marshallmallows marshallmallows force-pushed the T17056_add_l_for_gas_ls branch from 4d9c510 to c1702c7 Compare July 29, 2021 06:10
@marshallmallows marshallmallows merged commit 75952c9 into Graviti-AI:main Jul 29, 2021
@marshallmallows marshallmallows deleted the T17056_add_l_for_gas_ls branch July 29, 2021 06:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@linjiX linjiX linjiX approved these changes

@Lee-000 Lee-000 Lee-000 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@marshallmallows @coveralls @Lee-000 @linjiX