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

core: Add ruff rules D (docstring) #29406

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cbornet
Copy link
Collaborator

@cbornet cbornet commented Jan 24, 2025

This ensures that the code is properly documented: https://docs.astral.sh/ruff/rules/#pydocstyle-d

Copy link

vercel bot commented Jan 24, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview Jan 25, 2025 11:28am

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jan 24, 2025
@cbornet cbornet force-pushed the ruff-core-some-docs-2 branch 4 times, most recently from d7d9134 to c69d3b5 Compare January 24, 2025 11:24
@cbornet cbornet changed the title core: Add ruff rules D102 (missing docstring) core: Add ruff rules D (docstring) Jan 24, 2025
@cbornet cbornet closed this Jan 25, 2025
@cbornet cbornet force-pushed the ruff-core-some-docs-2 branch from c0ce268 to dbb6b7b Compare January 25, 2025 00:12
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Jan 25, 2025
@cbornet cbornet reopened this Jan 25, 2025
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Jan 25, 2025
@cbornet cbornet force-pushed the ruff-core-some-docs-2 branch 6 times, most recently from a8dea9b to c120449 Compare January 25, 2025 11:02
@cbornet cbornet force-pushed the ruff-core-some-docs-2 branch from c120449 to 23dc2bb Compare January 25, 2025 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:L This PR changes 100-499 lines, ignoring generated files.
Projects
Status: Triage
Development

Successfully merging this pull request may close these issues.

1 participant