Skip to content

fix one help-text typo#3268

Merged
talsperre merged 1 commit into
Netflix:masterfrom
LuisJG8:fix/run-cmds-help-typo-only
Jun 13, 2026
Merged

fix one help-text typo#3268
talsperre merged 1 commit into
Netflix:masterfrom
LuisJG8:fix/run-cmds-help-typo-only

Conversation

@LuisJG8

@LuisJG8 LuisJG8 commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

PR Type

  • Bug fix

Summary

Fix a missing space in the run --namespace help text so the example renders as empty string. --namespace= instead of empty string.--namespace=.

@greptile-apps

greptile-apps Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Fixes a missing space in the --namespace option help text in run_cmds.py. Without the space, the sentence "...with an empty string." ran directly into "--namespace=", making the rendered CLI help unreadable.

  • Adds a single leading space so the example (--namespace=) is separated from the preceding sentence in the help output.

Confidence Score: 5/5

Safe to merge — the change touches only a CLI help string with no logic impact.

Single-character whitespace correction in a help text string; no runtime logic, no tests, and no API surface are affected.

No files require special attention.

Important Files Changed

Filename Overview
metaflow/cli_components/run_cmds.py Adds a missing leading space to the --namespace option help text so the example renders as ...with an empty string. --namespace= instead of ...with an empty string.--namespace=

Reviews (1): Last reviewed commit: "fix one help-text typo" | Re-trigger Greptile

@talsperre
talsperre self-requested a review June 13, 2026 00:38
@talsperre
talsperre merged commit 38797e3 into Netflix:master Jun 13, 2026
39 checks passed
@codecov

codecov Bot commented Jun 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (master@08b7218). Learn more about missing BASE report.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #3268   +/-   ##
=========================================
  Coverage          ?   28.81%           
=========================================
  Files             ?      381           
  Lines             ?    52513           
  Branches          ?     9266           
=========================================
  Hits              ?    15130           
  Misses            ?    36349           
  Partials          ?     1034           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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