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

feat: shell-style read #533

Open
wants to merge 4 commits into
base: master
from
Open

feat: shell-style read #533

wants to merge 4 commits into from

Conversation

@leoj3n
Copy link

leoj3n commented Jan 29, 2019

lib/config.js Outdated Show resolved Hide resolved
@lorenwest
Copy link
Owner

lorenwest commented Jan 30, 2019

This looks like a healthy addition. I suggest you add an opt-in flag for this so people can take advantage of the functionality right away. Otherwise we'll have to wait until a major release - or make this feature a major release.

Best to have an opt-in flag, keep this major revision, then document the flag will change to opt-out in a future major revision.

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

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.