-
Notifications
You must be signed in to change notification settings - Fork 111
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
Consider merge up to https://github.com/rust-unofficial/patterns #7
Comments
Hey, I'm one of the maintainers of said repository and I think it would make sense to "merge" this repository as we are working on bringing |
it's ok. but how to do this, copy all code to rust-unoffical/patterns ? |
Hmm, as we have CI checks in place and we (maintainers) and others want to read over what is being merged into the repository it might be best, if we (I can help you there) open PRs for each pattern. I know it sounds like a lot of work, but we'll manage and it will be for the good. :) Also another question is the differences in licensing. If you agree I can start creating PRs for each of the patterns. :) |
I've reorganized the structure of the |
We should also be aware, that some Patterns might need changes, as we follow our |
@lpxxn Some of the patterns here felt like directly taken from another language which does not seeemed fit so well in rust which is probably the reason why it wasn't included in the rust patterns at first. One reason could be ergonomic. We should think twice before including all of them, maybe some might be good like State (although the example isn't good), but some others looked a bit weird (lots of Imagine having lifetime, borrowing, ownership pattern in java, what would happened? @simonsan Please try to make sure we have a good example or even better, it is a pattern being used in a popular crate, then I would say it is good to merge it, otherwise it feels weird to have a hard direct port. |
@lpxxn Thank you :) The point is, sometimes people just miss examples to write the corresponding articles, I did it now in the way, that I opened an issue for every missing article that we find examples in here and used them. My hope is, that more people write some articles about these patterns in the future because of the reason that they easily found an example for it, that they could still improve, if they wanted to. So @lpxxn @pickfire don't worry, we won't merge it into the repository immediately. But as examples to discuss about and improve upon together they are really nice. |
No description provided.
The text was updated successfully, but these errors were encountered: