-
Notifications
You must be signed in to change notification settings - Fork 383
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
Inspiration for more patterns #35
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
Comments
awesome, thanks! |
Yeah, we don't want to overlap with the rust api guidelines (which is also linked in the killercup article). Let's discuss each of those individually:
|
Should we close this in favour of #116 ? I copied everything needed from this issue over. |
In that issue the fact that these issues should be taken from that blog post is not clear in my opinion. |
Agreed, discussions should still take place in smaller issues, it's an umbrella. i thought about opening an issue for each of the points in this issue actually. hence why i asked about closing. we could link from each of these issues to this one here to make clear, that there might be material taken from it |
What you are suggesting is the opposite: too small issues 😂 |
Yeah, or even better use the discussion board for discussing how to write the patterns and keep the issues clean for actual issues, what do you think? |
It depends for what you mean with "actual issues". I think "discussions" are still in beta and they are more meant for "stack overflow" kind of random questions. |
Yeah, let's see how it develops. |
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
I recently started collecting some API design patterns here. I'm not sure if these match what this repo is all about (or which ones you already cover), but I just wanted to say: Feel free to take anything you like from that post! :)
(The markdown source of that post is here and I hereby relicense the content as MPL2 (in addition to CC-BY) and allow you to use it in this repo.)
The text was updated successfully, but these errors were encountered: