Skip to content

Conversation

@ndonkoHenri
Copy link
Contributor

@ndonkoHenri ndonkoHenri commented Oct 18, 2025

Summary by Sourcery

Streamline the CI workflow by consolidating Flet extension builds into a single job, merging Flutter analysis and Python packaging steps, and refining the publish and artifact upload configurations.

CI:

  • Combine all Flet extension builds into a single CI job with a loop instead of a matrix strategy
  • Merge Flutter analysis and Python package builds into one step for all extensions
  • Standardize the upload artifact step to produce a single flet-python-extensions artifact
  • Configure setup-uv to ignore an empty workdir and clear the cache dependency glob
  • Add a dry-run flag to the uv publish command in the PyPI publication step

@ndonkoHenri ndonkoHenri requested a review from Copilot October 18, 2025 13:22
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.

We've reviewed this pull request using the Sourcery rules engine

Copy link
Contributor

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 consolidates the building of Flet extension packages from multiple parallel jobs into a single sequential job to simplify the CI workflow.

  • Removed the matrix strategy that built each extension in a separate job
  • Combined all extension processing into a loop within one job step
  • Updated artifact naming from individual packages to a single combined artifact

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

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 18, 2025

Deploying flet-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 292c757
Status: ✅  Deploy successful!
Preview URL: https://17f9afa0.flet-docs.pages.dev
Branch Preview URL: https://improve-ci.flet-docs.pages.dev

View logs

@FeodorFitsner FeodorFitsner merged commit b363cee into main Oct 19, 2025
18 of 20 checks passed
@FeodorFitsner FeodorFitsner deleted the improve-ci branch October 19, 2025 01:02
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.

3 participants