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 box_patterns for stable rust 1.23 #44

Merged
merged 1 commit into from
Feb 12, 2018
Merged

Remove box_patterns for stable rust 1.23 #44

merged 1 commit into from
Feb 12, 2018

Conversation

ember-rose
Copy link
Contributor

As far as I can tell, the fluent crate is not using box_patterns. I removed the feature declaration, and after projectfluent/fluent-langneg-rs#3 is merged, we should be able to get this building on stable rust 1.23, resolving #42.

@stasm
Copy link
Contributor

stasm commented Jan 24, 2018

Ah, great! It looks like we removed the use of box destructuring in 66a6719#diff-caee7c45e15d65714d3976d059d7ccb2L102. Thanks for the PR!

@ember-rose
Copy link
Contributor Author

Thanks for the review! It looks like the Travis failure will be resolved after projectfluent/fluent-langneg-rs#3 is merged and the version get bumped.

@stasm
Copy link
Contributor

stasm commented Jan 24, 2018

Right :) I'll let @zbraniecki review projectfluent/fluent-langneg-rs#3 and then we can move forward here. Thanks again!

@stasm
Copy link
Contributor

stasm commented Jan 31, 2018

@zbraniecki When do you plan to publish a new release of fluent-locale-rs?

@zbraniecki
Copy link
Collaborator

Hmm, didn't have any strict plan yet. Wanted to wait for us to be ready to release new fluent-rs and release both alongside, but if you see a value in a release soon, I can do that!

@stasm
Copy link
Contributor

stasm commented Jan 31, 2018

If you feel comfortable bumping it to 0.3.1 we could then go ahead and merge this PR. It's a small change which makes it easier to get started with our code.

@zbraniecki zbraniecki merged commit 3ecd8cc into projectfluent:master Feb 12, 2018
@zbraniecki
Copy link
Collaborator

Thank you for your contributions @ashley-rose ! I released fluent-locale-rs 0.3.1 and merged your patch here and fluent works with stable!

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.

3 participants