Skip to content

✨ Add typer-slim package without extras, make typer include typer-slim[default] and integrate Typer CLI (typer command) into Typer#780

Merged
tiangolo merged 25 commits intomasterfrom
typer-slim
Mar 30, 2024
Merged

✨ Add typer-slim package without extras, make typer include typer-slim[default] and integrate Typer CLI (typer command) into Typer#780
tiangolo merged 25 commits intomasterfrom
typer-slim

Conversation

@tiangolo
Copy link
Member

✨ Add typer-slim package without extras, make typer include typer-slim[default] and integrate Typer CLI (typer command) into Typer

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@tiangolo tiangolo added the feature New feature, enhancement or request label Mar 30, 2024
@github-actions
Copy link
Contributor

@tiangolo tiangolo marked this pull request as ready for review March 30, 2024 02:08
@tiangolo tiangolo merged commit 7d5d6bb into master Mar 30, 2024
@tiangolo tiangolo deleted the typer-slim branch March 30, 2024 02:09
name = "typer"
# pip install typer-slim: is only the Typer library, without extras
# pip install typer: is typer-slim[standard], with extras and typer-cli
name = "typer-slim"

Choose a reason for hiding this comment

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

Perhaps you want to update the documentation to let users know that typer actually includes typer-slim and other packages?
This PR offuscates that and this change breaks users of FawltyDeps. We had declared typer as a dependency (-> failed), but it's now the project typer-slim the one that has a typer package.

Copy link
Member

Choose a reason for hiding this comment

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

Hi @FCamborda! Could you open an discussion thread with this question? Because I worry that it might otherwise get lost on this merged PR 🙏

Choose a reason for hiding this comment

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

@svlandeg Sure! Let me know if this needs improvement: #786

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature, enhancement or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants