Skip to content

Make streamlit run with no args mean streamlit run streamlit_app.py#12599

Merged
sfc-gh-tteixeira merged 15 commits intodevelopfrom
default-run
Sep 25, 2025
Merged

Make streamlit run with no args mean streamlit run streamlit_app.py#12599
sfc-gh-tteixeira merged 15 commits intodevelopfrom
default-run

Conversation

@sfc-gh-tteixeira
Copy link
Copy Markdown
Contributor

@sfc-gh-tteixeira sfc-gh-tteixeira commented Sep 22, 2025

Describe your changes

Tired of writing streamlit run streamlit_app.py? I have the solution! Save yourself 17 keystrokes by just writing streamlit run. If a filename is not included, the streamlit_app.py is now implied.

GitHub Issue Link (if applicable)

None

Testing Plan

  • Explanation of why no additional tests are needed
  • Unit Tests (JS and/or Python)
  • E2E Tests
  • Any manual testing needed?

Contribution License Agreement

By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

@snyk-io
Copy link
Copy Markdown
Contributor

snyk-io bot commented Sep 22, 2025 •
edited
Loading

🎉 Snyk checks have passed. No issues have been found so far.

✅ security/snyk check is complete. No issues have been found. (View Details)

✅ license/snyk check is complete. No issues have been found. (View Details)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 22, 2025 •
edited
Loading

✅ PR preview is ready!

Name Link
📦 Wheel file https://core-previews.s3-us-west-2.amazonaws.com/pr-12599/streamlit-1.50.0-py3-none-any.whl
🕹️ Preview app pr-12599.streamlit.app (☁️ Deploy here if not accessible)

graphite-app[bot]
graphite-app bot reviewed Sep 22, 2025
View reviewed changes
@bew
Copy link
Copy Markdown

bew commented Sep 22, 2025

It would be nice if we can give it a folder and the streamlit_app.py is automatically added

Our apps are currently run with:
streamlit run src/web_app/streamlit_app.py
It could be:
streamlit run src/web_app/
🤔

So streamlit run would be a shortcut for streamlit run .

@sfc-gh-tteixeira @graphite-app
Update lib/tests/streamlit/web/cli_test.py
5f70286 
Co-authored-by: graphite-app[bot] <96075541+graphite-app[bot]@users.noreply.github.com>
graphite-app[bot]
graphite-app bot reviewed Sep 22, 2025
View reviewed changes
@sfc-gh-tteixeira sfc-gh-tteixeira enabled auto-merge (squash) September 22, 2025 17:15
@sfc-gh-tteixeira
Copy link
Copy Markdown
Contributor Author

sfc-gh-tteixeira commented Sep 22, 2025

It would be nice if we can give it a folder and the streamlit_app.py is automatically added

Nice idea! However,in the past we considered making streamlit run some/path have a different meaning, related to multipage apps, so I worry about taking that ability away without spending some time considering the consequences. I'll run this by the team and see if they have any concerns.

Might be a separate PR, though!

@jrieke
Copy link
Copy Markdown
Collaborator

jrieke commented Sep 22, 2025

Cool idea, love it!

However,in the past we considered making streamlit run some/path have a different meaning, related to multipage apps, so I worry about taking that ability away without spending some time considering the consequences.

Hm I don't think there's a chance we'll do this anymore, given that we already have multipage apps v1 and v2 (and probably don't want a v3 :D), plus given that this would require changes in Cloud + SiS. So I'm fine adding that if you want to!

graphite-app[bot]
graphite-app bot reviewed Sep 22, 2025
View reviewed changes
@sfc-gh-bnisco sfc-gh-bnisco added security-assessment-completed change:feature PR contains new feature or enhancement implementation impact:users PR changes affect end users labels Sep 22, 2025
graphite-app[bot]
graphite-app bot reviewed Sep 23, 2025
View reviewed changes
graphite-app[bot]
graphite-app bot reviewed Sep 23, 2025
View reviewed changes
sfc-gh-lwilby
sfc-gh-lwilby approved these changes Sep 25, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@sfc-gh-lwilby sfc-gh-lwilby left a comment

Choose a reason for hiding this comment

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

LGTM

@sfc-gh-tteixeira sfc-gh-tteixeira merged commit 43c92cc into develop Sep 25, 2025
36 of 37 checks passed
@sfc-gh-tteixeira sfc-gh-tteixeira deleted the default-run branch September 25, 2025 10:42
@sfc-gh-amiribel sfc-gh-amiribel mentioned this pull request Oct 28, 2025
Closed
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@graphite-app graphite-app[bot] graphite-app[bot] left review comments

@sfc-gh-lwilby sfc-gh-lwilby sfc-gh-lwilby approved these changes

Assignees

No one assigned

Labels

change:feature PR contains new feature or enhancement implementation impact:users PR changes affect end users

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@sfc-gh-tteixeira @bew @jrieke @sfc-gh-lwilby @sfc-gh-bnisco