search

TheDataStation / ver

Data Discovery Tools and Systems
MIT License
6 stars 10 forks source link

Refine QuickStart #50

Closed zaki-indra closed 1 year ago

zaki-indra commented 1 year ago

This PR resolve #47.

Here are the few things included in this PR:

kevindharmawan commented 1 year ago

I think what #47 wants is to edit the quickstart.md to run Ver only via ver_cli.py, for example this block in the old quickstart.md:

cd dindex_builder
python3 dindex_builder.py --profile_data_path ../ddprofiler/output_profiles_json --force

Will become:

python3 ver_cli.py build_dindex ../ddprofiler/output_profiles_json

CMIIW

zaki-indra commented 1 year ago

Well, that is another way to interpret issue #47. I thought rewriting it meant using the ver_cli.py module to run the whole QuickStart procedure in a single CLI command. Anyway, I can add a new markdown that uses the ver_cli.py CLI if needed. What do you think, @snowgy, @kevindharmawan?

snowgy commented 1 year ago

@zaki-indra Kevin's suggestion is the original intention of #47. Could you add a new markdown that usesver_cli.py?