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

docs: add strategy using dedicated (un)structure method #396

Closed
wants to merge 1 commit into from

Conversation

pohlt
Copy link
Contributor

@pohlt pohlt commented Jul 19, 2023

Feel free to edit. 👍🏼

@Tinche
Copy link
Member

Tinche commented Jul 25, 2023

Howdy,

good job. We've had a small misunderstanding though; strategies are basically functions you can import and call, and they apply a potentially complex customization on your behalf. What you have here is a piece of documentation that looks like a recipe.

That's also good! It should just go elsewhere. Can you move it to the common usage examples page instead?

@pohlt
Copy link
Contributor Author

pohlt commented Jul 27, 2023

Ok, got it. Will take some time, but I'll do as requested.

@pohlt
Copy link
Contributor Author

pohlt commented Aug 21, 2023

Is included in #405, so closing here.

@pohlt pohlt closed this Aug 21, 2023
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

Successfully merging this pull request may close these issues.

2 participants