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

Add flet if no dependencies provided for flet publish command #4507

Closed
wants to merge 5 commits into from

Conversation

FeodorFitsner
Copy link
Contributor

@FeodorFitsner FeodorFitsner commented Dec 6, 2024

Fix #4493

Summary by Sourcery

Bug Fixes:

  • Ensure the flet publish command includes flet as a dependency if no other dependencies are provided.

ndonkoHenri and others added 5 commits December 3, 2024 13:11
# `Fixed` when I get no markers key in the version_value variable.

```bash
Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.12.5/x64/bin/flet", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/opt/hostedtoolcache/Python/3.12.5/x64/lib/python3.12/site-packages/flet/cli.py", line 12, in main
    flet_cli.cli.main()
  File "/opt/hostedtoolcache/Python/3.12.5/x64/lib/python3.12/site-packages/flet_cli/cli.py", line 89, in main
    args.handler(args)
  File "/opt/hostedtoolcache/Python/3.12.5/x64/lib/python3.12/site-packages/flet_cli/commands/build.py", line 557, in handle
    self.package_python_app()
  File "/opt/hostedtoolcache/Python/3.12.5/x64/lib/python3.12/site-packages/flet_cli/commands/build.py", line 1303, in package_python_app
    toml_dependencies = get_poetry_dependencies(
                        ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/hostedtoolcache/Python/3.12.5/x64/lib/python3.12/site-packages/flet_cli/utils/project_dependencies.py", line 41, in get_poetry_dependencies
    format_dependency_version(dependency, version)
  File "/opt/hostedtoolcache/Python/3.12.5/x64/lib/python3.12/site-packages/flet_cli/utils/project_dependencies.py", line 21, in format_dependency_version
    if version_value["markers"]:
       ~~~~~~~~~~~~~^^^^^^^^^^^
KeyError: 'markers'
```
* delete deprecations on py end

* delete deprecations on dart end
* remove minimum from SafeArea

* remove deprecated props
@FeodorFitsner FeodorFitsner changed the base branch from main to 0.25.2 December 6, 2024 18:33
@FeodorFitsner FeodorFitsner changed the base branch from 0.25.2 to main December 6, 2024 18:34
@FeodorFitsner FeodorFitsner changed the base branch from main to 0.25.2 December 6, 2024 18:37
@FeodorFitsner FeodorFitsner deleted the feodor/flet-publish-fix branch December 6, 2024 18:41
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.

flet publish creates broken website if no requirements.txt or pyproject.toml found
3 participants