📝 Add docstrings to docs/mkdocs-site#42
Conversation
Docstrings generation was requested by @petems. * #41 (comment) The following files were modified: * `src/mcp_github_pr_review_spec_maker/cli.py` * `src/mcp_github_pr_review_spec_maker/server.py` * `tests/test_enterprise_url_support.py` * `tests/test_git_pr_resolver.py` * `tests/test_mcp_server_tools.py` * `tests/test_rest_error_handling.py`
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
Important Review skippedCodeRabbit bot authored PR detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
|
@coderabbitai run the ruff linter on these changes and then commit |
|
Tip For best results, initiate chat on the files or code changes. 🧩 Analysis chain🏁 Script executed: Length of output: 83971 @petems I've run the ruff linter on the changes and found several issues that need to be fixed. However, I cannot commit code directly to the repository, even when requested. Issues FoundRuff Check: 138 errors (137 remaining)
Ruff Format: All 6 modified files need reformatting How to FixYou can fix these issues by running the following commands locally: # Auto-fix formatting issues
ruff format src/mcp_github_pr_review_spec_maker/cli.py \
src/mcp_github_pr_review_spec_maker/server.py \
tests/test_enterprise_url_support.py \
tests/test_git_pr_resolver.py \
tests/test_mcp_server_tools.py \
tests/test_rest_error_handling.py
# Check for any remaining issues
ruff check --fix src/mcp_github_pr_review_spec_maker/cli.py \
src/mcp_github_pr_review_spec_maker/server.py \
tests/test_enterprise_url_support.py \
tests/test_git_pr_resolver.py \
tests/test_mcp_server_tools.py \
tests/test_rest_error_handling.py
# Then commit the changes
git add -u
git commit -m "Fix ruff linting issues"
git pushThe main issues are long docstring lines that need to be wrapped to stay within the 88-character limit, and some mixed spaces/tabs in indentation that |
Docstrings generation was requested by @petems.
The following files were modified:
src/mcp_github_pr_review_spec_maker/cli.pysrc/mcp_github_pr_review_spec_maker/server.pytests/test_enterprise_url_support.pytests/test_git_pr_resolver.pytests/test_mcp_server_tools.pytests/test_rest_error_handling.pyThese files were kept as they were
tests/conftest.pytests/test_config_edge_cases.pytests/test_integration.pytests/test_status_fields.pyThese file types are not supported
AGENTS.mdCLAUDE.mdREADME.mdUV_COMMANDS.mddocs/architecture/overview.mddocs/changelog.mddocs/contributing/index.mddocs/getting-started/configuration.mddocs/getting-started/installation.mddocs/getting-started/quickstart.mddocs/guides/editor-integrations.mddocs/guides/local-stdio.mddocs/guides/remote-uv-endpoint.mddocs/index.mddocs/reference/cli.mddocs/reference/environment.mddocs/reference/mcp-manifest.mddocs/reference/publishing.mddocs/reference/tools.mddocs/security/index.mdmkdocs.ymlprompts/GITHUB_API_ROBUSTNESS.mdpyproject.tomlrequirements.txtspecs/PHASE2_ENTERPRISE_GITHUB_URLS.mdsrc/mcp_github_pr_review_spec_maker/mcp.jsonℹ️ Note