Skip to content

Make the --ssl-capath CLI argument a directory#1588

Open
rolandwalker wants to merge 1 commit intomainfrom
RW/make-capath-cli-arg-a-path
Open

Make the --ssl-capath CLI argument a directory#1588
rolandwalker wants to merge 1 commit intomainfrom
RW/make-capath-cli-arg-a-path

Conversation

@rolandwalker
Copy link
Contributor

Description

This influences the helpdoc, which now will show

--ssl-capath DIRECTORY

Checklist

  • I added this contribution to the changelog.md file.
  • I added my name to the AUTHORS file (or it's already there).
  • To lint and format the code, I ran
    uv run ruff check && uv run ruff format && uv run mypy --install-types .

@rolandwalker rolandwalker self-assigned this Feb 20, 2026
This influences the helpdoc, which now will show

    --ssl-capath DIRECTORY
@rolandwalker rolandwalker force-pushed the RW/make-capath-cli-arg-a-path branch from 2c38b00 to 5b219fc Compare February 20, 2026 11:54
@github-actions
Copy link

Findings
No correctness or security issues found in the change.

Missing Tests / Edge Cases

  1. Add a CLI test to assert --ssl-capath rejects files and non-existent paths and accepts existing directories. This prevents regressions in validation behavior. Reference: mycli/main.py:1666.

Notes

  • The help text showing DIRECTORY is consistent with Click’s directory type name.

If you want, I can draft the test.

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.

1 participant

Comments