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

Error if default config is missing *and* command requires config #100

Open
cisaacstern opened this issue Sep 1, 2023 · 1 comment
Open

Comments

@cisaacstern
Copy link
Member

#88 covers the case of an explicitly passed config that is missing.

It does not fix the case of a missing default config ("pangeo_forge_runner_config.py").

This is by design, because some commands (e.g., expand_meta) don't require config, so we should allow a missing default config to pass through without an error.

But what if a command does require config (e.g., bake) and the default config is missing? Let's resolve that in a separate PR. 🙂

@cisaacstern cisaacstern changed the title Error if default config is missing _and_ command requires config Error if default config is missing *and* command requires config Sep 1, 2023
@cisaacstern
Copy link
Member Author

xref #81 which is closely related

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant