-
Notifications
You must be signed in to change notification settings - Fork 42
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
Add keyword indexing + constructor #177
Conversation
This seems like a nice improvement in usability. Let's give folks a couple more days to express concerns, and then I'd favor merging. |
Excellent. Should I have a go at updating the readme to use this? |
Given the lack of expressed concerns, sure, that would be great! Note that in addition to the README itself there is a |
OK, have added the keyword indexing lines to I had a go at inserting a paragraph explaining that you can now use keywords, at the top of the readme. It's not great but see what you think? |
Test failure was my fault, in that JuliaMath/IntervalSets.jl#57 causes |
Thanks for a lovely PR, @mcabbott! |
This adds the following behaviour:
I think this was discussed somewhere.... only this comment perhaps?
Edit: this also closes #174, because you can write