add repos subcommand [continue #44] (#3) #65

Merged
lunny merged 23 commits from 6543/tea:repos_subcommand into master 3 years ago
6543 commented 3 years ago
Collaborator

continue #44
close #3

continue #44 close #3
6543 changed title from add repos subcommand [continue #44] (#3) to [WIP] add repos subcommand [continue #44] (#3) 3 years ago
6543 changed title from [WIP] add repos subcommand [continue #44] (#3) to add repos subcommand [continue #44] (#3) 3 years ago
6543 added the
kind/feature
label 3 years ago
6543 added this to the v0.2.0 milestone 3 years ago
6543 self-assigned this 3 years ago
Poster
Collaborator

@lunny do you know why CI is failing?

@lunny do you know why CI is failing?
zeripath approved these changes 3 years ago
Dismissed
jonasfranz reviewed 3 years ago
Dismissed
"Name",
"Type/Mode",
"SSH-URL",
"Owner",

Maybe use simpler headers like "Type" or "URL" since those are used as yaml keys too.

Maybe use simpler headers like "Type" or "URL" since those are used as yaml keys too.
6543 commented 3 years ago
Poster
Collaborator

Type is ok

about the SSH-URL, since i piced an old PR up to bring it upstream ... i realy have no idear why @root360-AndreasUlm add this colum. and URL is not the right way to describe it because it returns the ssh clone url - i change it to SSH

example output:

+------------------------------+--------+------------------------------------------------+-------------------+
|             NAME             |  TYPE  |                      SSH                       |       OWNER       |
+------------------------------+--------+------------------------------------------------+-------------------+
| 6543/go-sdk                  | fork   | git@gitea.com:6543/go-sdk.git                  |              6543 |
| 6543/GitNex                  | fork   | git@gitea.com:6543/GitNex.git                  |              6543 |
| 6543/markdownTest            | source | git@gitea.com:6543/markdownTest.git            |              6543 |
...

one thing I think about is to change "Name" values to repo names only because you get the whole "name" by combining Owner and Name ...

Type is ok about the SSH-URL, since i piced an old PR up to bring it upstream ... i realy have no idear why @root360-AndreasUlm add this colum. and URL is not the right way to describe it because it returns the ssh clone url - i change it to SSH example output: ``` +------------------------------+--------+------------------------------------------------+-------------------+ | NAME | TYPE | SSH | OWNER | +------------------------------+--------+------------------------------------------------+-------------------+ | 6543/go-sdk | fork | git@gitea.com:6543/go-sdk.git | 6543 | | 6543/GitNex | fork | git@gitea.com:6543/GitNex.git | 6543 | | 6543/markdownTest | source | git@gitea.com:6543/markdownTest.git | 6543 | ... ``` one thing I think about is to change "Name" values to repo names only because you get the whole "name" by combining Owner and Name ...

@6543:
The intention of the clone-URL was that you can cut the URL out of the table and directly use as parameter for git-clone.
The use-case for that is that you list all repository forks you own and clone those into a location.
Idea of sample command:
tea repos ls --mode fork --user my_user --output simple | cut -d' ' -f3 | xargs git clone

@6543: The intention of the clone-URL was that you can cut the URL out of the table and directly use as parameter for git-clone. The use-case for that is that you list all repository forks you own and clone those into a location. Idea of sample command: `tea repos ls --mode fork --user my_user --output simple | cut -d' ' -f3 | xargs git clone`
6543 commented 3 years ago
Poster
Collaborator

It will stay as SSH

It will stay as SSH
Poster
Collaborator

@jonasfranz can you review?

@jonasfranz can you review?
jonasfranz approved these changes 3 years ago
Dismissed
lunny closed this pull request 3 years ago
6543 deleted branch repos_subcommand 3 years ago
6543 modified the milestone from v0.2.0 to v0.1.0 3 years ago
continuous-integration/drone/pr Build is passing
The pull request has been merged as 5b3c92ee11.
Sign in to join this conversation.
Loading…
There is no content yet.