-
Notifications
You must be signed in to change notification settings - Fork 456
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
fix(docs): python docs for aspect #3002
Conversation
Hi @gertzakis 👋 Thank you for the PR 👍 Unfortunately, the source of truth for these snippets is in this file and your changes in this PR would be overridden the next time we run You can add your changes over there and use the |
Hello @ansgarm, thank you for clarifying this. I changed it, please check again :) |
4816412
to
5a5cb26
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @gertzakis 💯
I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you've found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Related issue
Fixes #
Description
I noticed that the documentation for Aspects in Python is not so easily understood. The reason is that it should call the Aspect class with the stack instance.
Checklist