Skip to content

Conversation

@safoinme
Copy link
Contributor

Describe changes

I implemented/fixed _ to achieve _.

Pre-requisites

Please ensure you have done the following:

  • I have read the CONTRIBUTING.md document.
  • I have added tests to cover my changes.
  • I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.
  • IMPORTANT: I made sure that my changes are reflected properly in the following resources:
    • ZenML Docs
    • Dashboard: Needs to be communicated to the frontend team.
    • Templates: Might need adjustments (that are not reflected in the template tests) in case of non-breaking changes and deprecations.
    • Projects: Depending on the version dependencies, different projects might get affected.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Other (add details above)

znegrin and others added 30 commits July 30, 2025 11:18
🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Add missing parameter documentation and return values for CLI
display functions to resolve CI docstring validation errors.
This includes fixes for service connectors, table utilities,
tags, pipelines, stack components, models, artifacts, authorized
devices, secrets, and service accounts.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Complete the docstring fixes for artifact versions and ensure all
critical CLI table functions have proper parameter and return documentation.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Add missing Args and Returns sections to functions in model.py
- Fix docstring issues in stack_components.py flavor functions
- Add parameter documentation to code_repository.py
- Fix utils.py print_page_info docstring
- Complete user_management.py docstring documentation
- Add missing documentation to project.py functions

All darglint docstring validation errors are now resolved for CLI files.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Fix Page[T] vs List[Any] type mismatches in prepare_list_data calls
- Add type ignore comments for model_dump no-any-return issues
- Fix missing type annotations in user_management.py
- Handle object vs List[Any] issues with proper type ignore comments
- Ensure proper handling of paginated vs non-paginated responses

Resolves all remaining mypy errors that were causing CI failures.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Fix line formatting in project.py, service_accounts.py, tag.py, user_management.py
- Ensure consistent formatting across all modified CLI files

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions github-actions bot added the internal To filter out internal PRs and issues label Nov 29, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 29, 2025

Documentation Link Check Results

Absolute links check failed
There are broken absolute links in the documentation. See workflow logs for details
Relative links check passed
Last checked: 2025-12-01 10:30:01 UTC

@safoinme safoinme force-pushed the experiment/try-partial-selective-ci-tests branch from 4fc7ef4 to 7f448e6 Compare December 1, 2025 09:20
@safoinme safoinme closed this Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal To filter out internal PRs and issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants