Skip to content

fix(pyproject): correct dev dependency name and add CLI entry point#16

Merged
Einswilli merged 1 commit intoAllDotPy:masterfrom
brunobrown:fix-cli-entrypoint
Jun 12, 2025
Merged

fix(pyproject): correct dev dependency name and add CLI entry point#16
Einswilli merged 1 commit intoAllDotPy:masterfrom
brunobrown:fix-cli-entrypoint

Conversation

@brunobrown
Copy link
Contributor

@brunobrown brunobrown commented Jun 12, 2025

This PR fixes two issues in the pyproject.toml that prevent the CLI from working and break dependency resolution with uv.


What was fixed

1. 🔧 Invalid dependency name

  • Replaced buil (typo) with the correct build in [project.optional-dependencies.dev].
  • The typo caused the following error during installation:

2. 🚀 Missing CLI entry point

  • Added the [project.scripts] section to register the fletx CLI command:
[project.scripts]
fletx = "fletx.__main__:main"
  • This enables terminal execution like:
fletx new my_app --no-install
  • Without this, users encounter:
error: Failed to spawn: `fletx`
Caused by: No such file or directory (os error 2)

How to test

Here’s the exact procedure used to test this fix:

  • Option 1: With pip only

  1. cd /path/to/fletx

  2. pip install -e .

  3. fletx new my_app --no-install

✅ CLI should work as expected.


  • Option 2: With uv

  1. Run uv build in the root of the forked FletX project.

  2. Created a new Python project elsewhere.

  3. In that project’s pyproject.toml, added the dependency pointing to the local fork:

dependencies = [
    "fletx @ file:///path/to/your/local/fletx",
]
  1. Run uv sync to install all dependencies (including the local fletxr).

  2. Tested the CLI with:

uv run fletx new fletx_login_migration --no-install

✅ CLI should work as expected.


Let me know if you want me to adjust anything!
Thanks for this amazing project - I'm happy to contribute.

Copy link
Contributor

@Einswilli Einswilli left a comment

Choose a reason for hiding this comment

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

Thanks!
This looks cleaner🎉

@Einswilli Einswilli merged commit 03bdd4a into AllDotPy:master Jun 12, 2025
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.

2 participants