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-level options from i18n fixtures #2078

Merged
merged 1 commit into from
Nov 13, 2015

Conversation

floehopper
Copy link
Contributor

Since #2005, flow-level options are no longer used as a fallback, so we can
safely remove these.

Expected observable changes

  • None

@chrisroos
Copy link
Contributor

Looks good to me.

@chrisroos chrisroos self-assigned this Nov 13, 2015
@chrisroos chrisroos added the LGTM label Nov 13, 2015
Since #2005, flow-level options are no longer used as a fallback, so we can
safely remove these.
@floehopper floehopper force-pushed the remove-flow-level-options-from-i18n-fixtures branch from 3aaba72 to 85374b9 Compare November 13, 2015 11:05
@floehopper
Copy link
Contributor Author

I've rebased this against master and force-pushed in preparation for merging.

floehopper added a commit that referenced this pull request Nov 13, 2015
…-i18n-fixtures

Remove flow-level options from i18n fixtures
@floehopper floehopper merged commit 4f2d7d9 into master Nov 13, 2015
@floehopper floehopper deleted the remove-flow-level-options-from-i18n-fixtures branch November 13, 2015 11:08
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.

2 participants