Conversation
* Replace Text with str * Replace alias "_Str" with str * Import from collections.abc and re where applicable * Remove Python 2 branches
stubs/PyYAML/METADATA.toml
Outdated
| @@ -1,2 +1 @@ | |||
| version = "5.4.*" | |||
| python2 = true | |||
| version = "6.0" | |||
There was a problem hiding this comment.
Do we really want to specify an exact version? pip install 6.0.0 works, and looking at the changelog, bugfix releases will likely be numbered like 6.0.x.
There was a problem hiding this comment.
Using 6.0.* won't work, because there is no matching version at the moment. We can update this, once 6.0.1 gets released.
There was a problem hiding this comment.
The following seems to work just fine for me, pip install 'pyyaml==6.0.*'
There was a problem hiding this comment.
That's weird. There were problems with using stars for non-existing parts in CI when doing the change to allowing star syntax. I'll try it here.
|
This introduces a bunch of new stubtest warnings, but they somehow don't occur in CI. Here's this PR, with pyyaml 6.0 (aka pyyaml 6.0.0, both seem to install the same thing): master with pyyaml 5.4.1: |
|
It's because they're allowlisted: https://github.com/python/typeshed/blob/master/stubs/PyYAML/@tests/stubtest_allowlist.txt allowlist dates back to the initial commit, which means they're allowlisted to have CI be green, rather than any especially good reason. |
|
I'll have a look. |
* Add yaml._yaml and move CParser and CEmitter there. * Add missing functions, classes, and arguments. * Use relative imports in some modules. * Add __all__ to yaml.cyaml. * Remove unnecessary noqa markers. * Update arguments.
hauntsaninja
left a comment
There was a problem hiding this comment.
Thanks, skimmed, seems fine!
No description provided.