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

Relay: do not enforce project wide configuration #203

Merged
merged 1 commit into from Dec 2, 2019

Conversation

mrtnzlml
Copy link
Member

@mrtnzlml mrtnzlml commented Dec 2, 2019

This feature was requested here #189 and was greatly improved in Babel itself here babel/babel#10778. It can however be annoying and sometimes difficult to do such modification so I changed my mind and I'd like to allow it after all. It's because this code where we require "upward" mode is necessary only for Universe development and should not affect users of our NPM package.

@tbergquist-godaddy
Copy link
Contributor

LGTM, but you have some merge conflicts here 😞

This feature was requested here #189 and was greatly improved in Babel itself here babel/babel#10778. It can however be annoying and sometimes difficult to do such modification so I changed my mind and I'd like to allow it after all. It's because this code where we require "upward" mode is necessary only for Universe development and should not affect users of our NPM package.
@mrtnzlml mrtnzlml force-pushed the relay-do-not-enforce-project-wide-config branch from c7c72c6 to a2716e5 Compare December 2, 2019 15:42
@mrtnzlml mrtnzlml merged commit a791727 into master Dec 2, 2019
@mrtnzlml mrtnzlml deleted the relay-do-not-enforce-project-wide-config branch December 2, 2019 15:46
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.

None yet

2 participants