organizations sorting does not work about sourced-ce

[smacker]

can't reproduce with using spaces as a separator:

$ go run cmd/sourced/main.go orgs init -t <token> go-chi golang-migrate
docker-compose working directory set to go-chi,golang-migrate
Creating network "srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_default" with the default driver
Creating volume "srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_gitbase_repositories" with default driver
Creating volume "srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_gitbase_indexes" with default driver
Creating volume "srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_metadata" with default driver
Creating volume "srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_postgres" with default driver
Creating volume "srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_redis" with default driver
Creating srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_redis_1      ... done
Creating srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_metadatadb_1 ... done
Creating srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_postgres_1   ... done
Creating srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_bblfsh_1     ... done
Creating srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_gitbase_1      ... done
Creating srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_bblfsh-web_1   ... done
Creating srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_ghsync_1       ... done
Creating srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_gitcollector_1 ... done
Creating srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_sourced-ui_1   ... done
smacker in ~/Work/sourced-ce on master [?]
$ go run cmd/sourced/main.go orgs init -t <token> golang-migrate go-chi
Stopping srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_sourced-ui_1 ... done
Stopping srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_gitbase_1    ... done
Stopping srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_bblfsh-web_1 ... done
Stopping srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_postgres_1   ... done
Stopping srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_bblfsh_1     ... done
Stopping srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_metadatadb_1 ... done
Stopping srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_redis_1      ... done
docker-compose working directory set to go-chi,golang-migrate
Starting srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_bblfsh_1     ... done
Starting srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_metadatadb_1 ... done
Starting srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_redis_1      ... done
Starting srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_postgres_1   ... done
Starting srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_gitbase_1      ... done
Starting srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_bblfsh-web_1   ... done
Starting srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_ghsync_1       ... done
Starting srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_gitcollector_1 ... done
Starting srcd-z28ty2hplgdvbgfuzy1tawdyyxrl_sourced-ui_1   ... done
$ go run cmd/sourced/main.go workdirs
* go-chi,golang-migrate

most probably carlosms-test-org,bblfsh is treated as 1 organization but we don't have validation.

We have 2 options:

• update code to change space to , as a separator (or support both)
• update docs

[carlosms]

that's interesting.

We have 2 options:

• update code to change space to , as a separator (or support both)
• update docs

Both are fine. I don't have a strong preference, but I have a feeling the comma-separated list can be more intuitive. I mean if we could and if it's not too complex, we could also accept a b/a,b/a, b...

[smacker]

Ok. I'll implement support for , in addition to current .

comma-separated list can be more intuitive

I feel the opposite, positional arguments are usually passed using space as a delimiter in all the cli tools I know.