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.

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

cli should not be dependent on a specific directory structure #1558

Open
corysimmons opened this issue Feb 4, 2019 · 2 comments
Open

cli should not be dependent on a specific directory structure #1558

corysimmons opened this issue Feb 4, 2019 · 2 comments
Assignees
Labels
a/collab-ci-cd c/cli Related to CLI e/hard Will take weeks of effort k/enhancement New feature or improve an existing feature p/medium non-urgent issues/features that are candidates for being included in one of the upcoming sprints s/triaged This has been reviewed by Hasura

Comments

@corysimmons
Copy link

To run $ hasura console I've needed to have a config.yaml present.

It only had the graphql endpoint in it.

If I delete this file $ hasura console fails.

If I delete this file and use the --endpoint flag (effectively replacing the only data config.yaml was providing), $ hasura console still fails. I'd expect it to work.

My use-case is wanting to do something like $ yarn dev:staging and $ yarn dev:production where these scripts just have different --endpoints.

@shahidhk shahidhk added k/enhancement New feature or improve an existing feature c/cli Related to CLI p/medium non-urgent issues/features that are candidates for being included in one of the upcoming sprints e/intermediate can be wrapped up in a week labels Feb 5, 2019
@shahidhk shahidhk changed the title Allow me to throw config.yaml in the trash cli should not be dependent on a specific directory structure Apr 26, 2019
@letientai299
Copy link

My issue (#2039) is closed as duplicated to this one, so I will plug my suggestion here.

  • Beside flag --endpoint, we can also have flag --admin-secret, and that will free us from config.yaml completely.

  • The folder migration is not needed for following commands:

    • metadata version
    • metadata clear
    • metadata export (the folders can be created on the fly)
    • console (with the help of 2 new flags)

@HarryVasanth
Copy link

My issue (#2039) is closed as duplicated to this one, so I will plug my suggestion here.

  • Beside flag --endpoint, we can also have flag --admin-secret, and that will free us from config.yaml completely.

  • The folder migration is not needed for following commands:

    • metadata version
    • metadata clear
    • metadata export (the folders can be created on the fly)
    • console (with the help of 2 new flags)

Did you manage to find a workaround for this @letientai299?

@manasag manasag added the s/triaged This has been reviewed by Hasura label Dec 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a/collab-ci-cd c/cli Related to CLI e/hard Will take weeks of effort k/enhancement New feature or improve an existing feature p/medium non-urgent issues/features that are candidates for being included in one of the upcoming sprints s/triaged This has been reviewed by Hasura
Projects
None yet
Development

No branches or pull requests

7 participants