Skip to content
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

Document the discussion in #33 #49

Closed
ceridwen opened this issue Sep 22, 2015 · 1 comment
Closed

Document the discussion in #33 #49

ceridwen opened this issue Sep 22, 2015 · 1 comment

Comments

@ceridwen
Copy link

When I went to try to figure out what the purpose of the adapter argument to wrapt.decorator, which I only found by reading the code, I eventually ended up reading through the closed issue because it wasn't otherwise documented. A brief summary, possibly with some of the examples used in the issue, would be useful somewhere in the docs, maybe under an "Advanced Usage" or similar heading?

@GrahamDumpleton
Copy link
Owner

Although the whole discussion wasn't transplanted to the docs, it does seem that details of the adapter factory were added, so going to close out this issue now. For more in depth discussion of background the other GitHub issue exists.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants