Skip to content
This repository has been archived by the owner on Jun 1, 2023. It is now read-only.

Add EnsureEnv helper to automate credential process #212

Merged
merged 4 commits into from
Jul 9, 2019
Merged

Conversation

tylerball
Copy link
Contributor

This PR introduces an EnsureEnv task that can be added as a prerequisite to commands to ensure that a .env file exists before the command runs. If it doesn't, it will ask the user for their credentials and create the file. This allows us to remove the .env requirement from the creation flow, along with the .env fixes in shopify_app v11.0.1 (Shopify/shopify_app#776).

Closes #156

@tylerball tylerball requested review from tanema and katiedavis July 4, 2019 19:18
@tylerball tylerball requested a review from a team as a code owner July 4, 2019 19:18
@davekilljoy davekilljoy mentioned this pull request Jul 5, 2019
7 tasks
Copy link
Contributor

@katiedavis katiedavis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎩Looks great to me. I like this flow a lot better!

@katiedavis katiedavis mentioned this pull request Jul 7, 2019
@tylerball tylerball merged commit fdb6618 into master Jul 9, 2019
@tylerball tylerball deleted the ensure-env branch July 9, 2019 18:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Split asking for credentials out from create command
3 participants