Skip to content

Conversation

ndonkoHenri
Copy link
Contributor

No description provided.

Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Sorry @ndonkoHenri, your pull request is larger than the review limit of 150000 diff characters

@ndonkoHenri ndonkoHenri requested a review from Copilot September 19, 2025 14:14
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates cross-references in docstrings across the Python SDK to use a shortened format. The changes replace full module paths like [flet.Control] with shortened versions like [flet.] and introduce [(c).] references for current class properties.

Key changes include:

  • Updating dependency versions in pyproject.toml to newer versions
  • Converting docstring cross-references from full paths to shortened format
  • Using [(c).] for references to current class members
  • Reorganizing some __all__ imports for better alphabetical ordering

Reviewed Changes

Copilot reviewed 121 out of 122 changed files in this pull request and generated 5 comments.

File Description
sdk/python/pyproject.toml Update dependency versions for documentation tools
sdk/python/packages/flet/src/flet/**/*.py Convert docstring cross-references to shortened format throughout the codebase

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

cloudflare-workers-and-pages bot commented Sep 19, 2025

Deploying flet-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: c2dcf2d
Status: ✅  Deploy successful!
Preview URL: https://774336ec.flet-docs.pages.dev
Branch Preview URL: https://v1-cross-refs.flet-docs.pages.dev

View logs

Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

New security issues found

Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

New security issues found

@ndonkoHenri ndonkoHenri marked this pull request as ready for review September 21, 2025 09:36
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Sorry @ndonkoHenri, your pull request is larger than the review limit of 150000 diff characters

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant