Skip to content

Document check_path_length to default to False#1036

Merged
jaimergp merged 2 commits into
conda:mainfrom
marcoesters:check-path-length
Sep 11, 2025
Merged

Document check_path_length to default to False#1036
jaimergp merged 2 commits into
conda:mainfrom
marcoesters:check-path-length

Conversation

@marcoesters
Copy link
Copy Markdown
Contributor

Description

The documentation states that check_path_length defaults to True. The code, however, indicates that the default value is None and has been None for a long time. So, the default behavior is False.

Correct the error in the documentation. Additionally, limit the type to bool since there is no special behavior for None.

Xref: #1022

Checklist - did you ...

  • Add a file to the news directory (using the template) for the next release's release notes?
  • Add / update necessary tests?
  • Add / update outdated documentation?

@github-project-automation github-project-automation Bot moved this to 🆕 New in 🔎 Review Aug 11, 2025
@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Aug 11, 2025
@marcoesters marcoesters changed the title Check path length Document check_path_length to default to False Aug 11, 2025
@marcoesters marcoesters marked this pull request as ready for review August 11, 2025 17:08
@marcoesters marcoesters requested a review from a team as a code owner August 11, 2025 17:08
@github-project-automation github-project-automation Bot moved this from 🆕 New to ✅ Approved in 🔎 Review Sep 11, 2025
@jaimergp jaimergp merged commit a630275 into conda:main Sep 11, 2025
23 checks passed
@github-project-automation github-project-automation Bot moved this from ✅ Approved to 🏁 Done in 🔎 Review Sep 11, 2025
@marcoesters marcoesters deleted the check-path-length branch November 11, 2025 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed [bot] added once the contributor has signed the CLA

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants