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

Remove flow option experimental.const_params #1160

Merged
merged 1 commit into from
Dec 17, 2017

Conversation

leebyron
Copy link
Contributor

@leebyron leebyron commented Dec 17, 2017

Merges and closes #1157

Codebases which don't enable this experiment show flow errors, so removing it from the project to avoid regression

Merges #1157

Codebases which don't enable this experiment show flow errors
@leebyron leebyron force-pushed the remove-experimental-const-params branch from 8bc06ca to de4272c Compare December 17, 2017 02:21
@leebyron leebyron merged commit 68ec835 into master Dec 17, 2017
@leebyron leebyron deleted the remove-experimental-const-params branch December 17, 2017 02:29
IvanGoncharov added a commit to IvanGoncharov/graphql-js that referenced this pull request Feb 8, 2018
leebyron pushed a commit that referenced this pull request Feb 8, 2018
* Update Flow to 0.65.0

* Replace '@flow' with '@flow strict'

* Partial revert of #1160

* lexicographicSortSchema: Remove workarounds for non-const args

* Reduce scope of change

* Reduce scope of change
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants