Skip to content
This repository has been archived by the owner on Jan 27, 2021. It is now read-only.

incomplete cli prototype #63

Closed
wants to merge 1 commit into from
Closed

incomplete cli prototype #63

wants to merge 1 commit into from

Conversation

butonic
Copy link
Member

@butonic butonic commented Jul 21, 2020

just a dump of an incompete accounts cli that uses the accounts api to create and update users

@update-docs
Copy link

update-docs bot commented Jul 21, 2020

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@butonic butonic requested a review from IljaN July 21, 2020 12:23
@butonic butonic added the enhancement New feature or request label Jul 21, 2020
@IljaN IljaN self-assigned this Jul 21, 2020
@IljaN IljaN force-pushed the accounts-cli branch 4 times, most recently from 307c8dc to 0d1ac33 Compare July 27, 2020 10:05
ocis-accounts [list|delete|update|add|inspect]

Implements UpdateMask for the update request. Changed server-handler accordingly.
The commands use service-discovery to discover the backend.
@ownclouders
Copy link

Codacy Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- pkg/command/list_accounts.go  2
- pkg/command/inspect_account.go  3
- pkg/service/v0/accounts.go  1
- pkg/flagset/flagset.go  1
- pkg/command/update_account.go  4
- pkg/command/remove_account.go  3
- pkg/command/add_account.go  2
         

Clones added
============
- pkg/command/inspect_account.go  1
- pkg/flagset/flagset.go  25
- pkg/command/remove_account.go  1
         

Clones removed
==============
+ pkg/service/v0/accounts.go  -5
         

See the complete overview on Codacy

@sonarcloud
Copy link

sonarcloud bot commented Jul 27, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@IljaN IljaN closed this Jul 27, 2020
@IljaN
Copy link
Member

IljaN commented Jul 27, 2020

Continued #69

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants