Skip to content

Handle error where .astro dir exist but config.yaml does not#77

Merged
andscoop merged 2 commits intomasterfrom
feature/6-friendlier-fail-if-config-does-not-exist
Aug 22, 2018
Merged

Handle error where .astro dir exist but config.yaml does not#77
andscoop merged 2 commits intomasterfrom
feature/6-friendlier-fail-if-config-does-not-exist

Conversation

@andscoop
Copy link
Contributor

This is based off of @benjamingregory 's error in #6. I was unable to replicate but he walked me through the error he was receiving. I believe this will resolve it, but unable to test on my machine.

@benjamingregory Once you are able to verify this solves the issue we can merge.

Scenario:
cd astro-cli && go build . && mkdir test && mv astro-cli test && cd test && ./astro-cli airflow init && rm .astro/config.yaml && ./astro-cli airflow start

Expected output:
Error: Project not initialized

@ryw
Copy link
Contributor

ryw commented Jul 7, 2018

@benjamingregory /bump

@ryw ryw added this to the v0.4 milestone Jul 20, 2018
@andscoop andscoop force-pushed the feature/6-friendlier-fail-if-config-does-not-exist branch from 1c73a29 to 22aaee1 Compare August 22, 2018 18:24
@andscoop andscoop merged commit e8c0d34 into master Aug 22, 2018
@andscoop andscoop deleted the feature/6-friendlier-fail-if-config-does-not-exist branch August 22, 2018 18:26
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

Successfully merging this pull request may close these issues.

3 participants