-
-
Notifications
You must be signed in to change notification settings - Fork 326
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
const
ify some of the methods
#137
Comments
Sure, if you want to PR that would be great |
Please do submit a PR for this! I'll review it when it's ready, and we can publish a new release after the just-merged #138 and your PR are both landed. |
I'll do that this evening. One thing to note is that while this isn't a breaking change, reverting it will be; I'll take care to mention any possible pain points in my PR. |
Yep, as long as we're only adding @Auburn I think the best way to handle versioning when faced with the many different languages is that you plan to keep the the |
I'm currently writing a voxel terrain generator (shush), and naturally I'm using perlin noise for that. Your crate has a very nice API, and thanks to its simplicity, converting most of the methods into
const
methods should be fairly straightforward.I'd be glad to PR this myself, if you'd like.
The text was updated successfully, but these errors were encountered: