Exclude tests and ensurepip from default frozen stdlib#6520
Exclude tests and ensurepip from default frozen stdlib#6520
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. Comment |
Co-authored-by: youknowone <69878+youknowone@users.noreply.github.com>
Co-authored-by: youknowone <69878+youknowone@users.noreply.github.com>
crates/pylib/Cargo.toml
Outdated
|
|
||
| [features] | ||
| freeze-stdlib = ["dep:rustpython-compiler-core", "dep:rustpython-derive"] | ||
| freeze-stdlib-full = ["freeze-stdlib"] |
There was a problem hiding this comment.
Change the default. we don't need this feature. blocklisted libs are useless
There was a problem hiding this comment.
Implemented: removed the freeze-stdlib-full feature and kept the default frozen stdlib excluding Lib/test and Lib/ensurepip (commit 5a5733a).
Co-authored-by: youknowone <69878+youknowone@users.noreply.github.com>
|
@copilot fix clippy warnings |
Co-authored-by: youknowone <69878+youknowone@users.noreply.github.com>
cargo clippy -p rustpython-derive-impl -- -D warnings)Original prompt
Lib/test/andLib/ensurepipfromfreeze-stdlib#4594💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.