Skip to content

Add docstrings for recipes#2185

Merged
KaelanDt merged 2 commits intomainfrom
kaelan/recipe-docstrings
Jun 4, 2025
Merged

Add docstrings for recipes#2185
KaelanDt merged 2 commits intomainfrom
kaelan/recipe-docstrings

Conversation

@KaelanDt
Copy link
Copy Markdown
Collaborator

@KaelanDt KaelanDt commented Jun 4, 2025

Before submitting
  • Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
  • Did you read the contributor guideline, Pull Request section?
  • Did you make sure to update the docs?
  • Did you write any new necessary tests?

What does this PR do?

Partially fixes #2114. Plugins coming in subsequent PR

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

cc @Borda @lantiga

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jun 4, 2025
Copy link
Copy Markdown
Collaborator

@t-vi t-vi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Who would not love docstrings.

@KaelanDt KaelanDt merged commit e7c0458 into main Jun 4, 2025
50 checks passed
@KaelanDt KaelanDt deleted the kaelan/recipe-docstrings branch June 4, 2025 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation lightning-l1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docstrings for recipes and plugins

2 participants