-
Notifications
You must be signed in to change notification settings - Fork 1
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
Fix docs #19
Conversation
I'm not convinced by the new interface name (neither by the package name btw). However, I prefer to postpone discussions on naming, once we have agreed on the interface definition and on the scope of this package. It's easier to name something, once we know what the thing is. So it's fine for me to keep the new wording for time being. +1 |
I suggest we don't say anything about "object with LV inside" other than pointing out this package just focus on LV and "particle" or whatever shouldn't be |
minor comments, can be merged once resolved. |
I removed the docs preview todo for now but opened #23 @mmikhasenko if you are happy with my comments, than I would like to merge ;-) |
Closes #14
Checklist