Skip to content

docs: improve cli description - #57

Open
estellebertrand wants to merge 11 commits into
mainfrom
docs/improve-cli-description
Open

docs: improve cli description#57
estellebertrand wants to merge 11 commits into
mainfrom
docs/improve-cli-description

Conversation

@estellebertrand

@estellebertrand estellebertrand commented May 21, 2026

Copy link
Copy Markdown
Contributor
  • Fix a typo in "human-friendly"[not a typo] Use a more natural phrasing
  • Unify phrasing syntax
  • Try to provide more syntax information by sub command in the perspective of a usage by AI-agent, to avoid them running loop of "--help" in case of no doc is passed in their context

Updated command descriptions for clarity and consistency.
Updated command descriptions to clarify required parameters and improve user guidance.
Updated command descriptions for clarity and consistency.
Updated command descriptions for clarity and consistency.
Updated command descriptions for clarity and consistency.
Refactor topic commands for improved clarity and consistency.
@estellebertrand
estellebertrand requested a review from Pierlou May 21, 2026 13:14

@Pierlou Pierlou left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just quibbling around 🤓

Comment thread datagouv/commands/dataset.py Outdated
set: list[str] = typer.Option([], "--set"),
) -> None:
"""Update a dataset. Each `--set` option is expected as `<key>=<new_value>`."""
"""Update a dataset by `id`. Each `--set` option is expected as `<key>=<new_value>`."""

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I find it weird to have this syntax here while we have "id required" above, maybe we'd rather standardize that too

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, i prefer the 'by id' version here, but i didn't know how to translate that in 'Display the main attributes of a dataset in a human-friendly format. id required.'

Maybe 'Display the main attributes of a dataset, looked up by id, in a human-friendly format.' but heavy

Comment thread datagouv/commands/dataset.py Outdated
estellebertrand and others added 2 commits May 21, 2026 15:43
Co-authored-by: Pierlou Ramade <48205215+Pierlou@users.noreply.github.com>
Co-authored-by: Pierlou Ramade <48205215+Pierlou@users.noreply.github.com>
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