Julien Palard added the comment:
One true positive today: https://github.com/python/cpython/pull/32355
--
___
Python tracker
<https://bugs.python.org/issue42
Julien Palard added the comment:
This is resolved since https://github.com/python/docsbuild-scripts/pull/28.
It could still happen in very specific conditions, but not for a long time, see:
https://github.com/python/docs-community/issues/41#issuecomment-1088462199=
--
nosy: +mdk
Julien Palard added the comment:
New changeset d0e696e05d4aaca1f1cde72a5c3326ca3d0f5c24 by Julien Palard in
branch 'main':
bpo-42238: [doc]: A make sucpicious false positive. (GH-32329)
https://github.com/python/cpython/commit/d0e696e05d4aaca1f1cde72a5c3326ca3d0f5c24
Change by Julien Palard :
--
pull_requests: +30388
pull_request: https://github.com/python/cpython/pull/32329
___
Python tracker
<https://bugs.python.org/issue42
Julien Palard added the comment:
New changeset bdc497496548e30fa208a8d98c30bf6d1833ac4c by Julien Palard in
branch 'main':
bpo-42238: [doc]: make suspicious: false positive. (GH-32292)
https://github.com/python/cpython/commit/bdc497496548e30fa208a8d98c30bf6d1833ac4c
Julien Palard added the comment:
One true positive and one false positive this week, see:
- 7f9c084fdec7ddcfe8855aa79f98545591ae2261
- ec8906fb5930b1f078e2a2170cdf445e6c6faf57
--
___
Python tracker
<https://bugs.python.org/issue42
Change by Julien Palard :
--
pull_requests: +30354
pull_request: https://github.com/python/cpython/pull/32292
___
Python tracker
<https://bugs.python.org/issue42
Julien Palard added the comment:
> Since you seem to be challenging the value of 9, my question to you
> is, why do you care what the refcount of 1 is?
Yesterday I was teaching Python, and we were speaking of integer immutability,
names being "labels to objects"
Julien Palard added the comment:
Hum, and why 9? I am probably missing something obvious but 1 should be
enough to ensure the value never hits 0. Except for refcount bugs obviously,
but I don't think this is the right reason
Julien Palard added the comment:
Since
https://github.com/python/cpython/commit/121f1f893a39d0b58d3d2b5597505c154ecaac2a,
`sys.getrefcount(1)` is surprising:
>>> __import__("sys").getrefcount(1)
100210
Should sys.getrefcount try to "fix" the value
Julien Palard added the comment:
New changeset ec8906fb5930b1f078e2a2170cdf445e6c6faf57 by Julien Palard in
branch 'main':
bpo-42238: [doc] Some lines moved in rst, but had hardcoded lineno in
susp-ignored.csv. (GH-32070)
https://github.com/python/cpython/commit
Change by Julien Palard :
--
pull_requests: +30159
pull_request: https://github.com/python/cpython/pull/32070
___
Python tracker
<https://bugs.python.org/issue42
Change by Julien Palard :
--
pull_requests: +30067
pull_request: https://github.com/python/cpython/pull/31977
___
Python tracker
<https://bugs.python.org/issue42
Julien Palard added the comment:
News:
Hitting 3 months without a true positive from `make suspicious`, looks like
sphinxlint starting to take over properly, I hope to close this issue soon
(like in a few months maybe).
Also sphinxlint attracted some contributors [1] \o/
[1] https
Julien Palard added the comment:
New changeset b878b3af0b3a9e3ab3ffcaf90a4066dfb3bb7cac by Julien Palard in
branch 'main':
bpo-42238: [doc] moving from rstlint.py to sphinx-lint. (GH-31097)
https://github.com/python/cpython/commit/b878b3af0b3a9e3ab3ffcaf90a4066dfb3bb7cac
Julien Palard added the comment:
Oh, it's ~unrelated, but thanks for the heads up, I overlooked a Sentry error :D
It's related to: https://github.com/python/docsbuild-scripts/issues/122, let's
track it there.
(The visible effects are the same: the full build, reponsible for deleting
Change by Julien Palard :
--
pull_requests: +29282
pull_request: https://github.com/python/cpython/pull/31097
___
Python tracker
<https://bugs.python.org/issue42
Julien Palard added the comment:
Georg, I was able to move forward:
- https://github.com/sphinx-contrib/sphinx-lint
- https://pypi.org/project/sphinx-lint/
Does everything looks good to you?
Next step would be to remove rstlint.py from Docs/tools and add a dependency to
rstlint from
Julien Palard added the comment:
OK, we have: https://github.com/sphinx-contrib/sphinx-lint
Ping me if you want rights :)
--
___
Python tracker
<https://bugs.python.org/issue42
Julien Palard added the comment:
> You gave me a small shock, but I checked and it was only 11/12 years ago :D
HAha! Either I misstyped 12 as 21, either I'm bad at math, sorry for the shock
;)
> I'm fine with moving this out of tree; please coordinate with the Sphinx
> maintainer
Julien Palard added the comment:
New changeset 44afdbd5af4503e376148e9404b9c7a4f595b1fe by Julien Palard in
branch 'main':
bpo-42238: [doc] Avoid hardcoding fast-moving lines in susp-ignored.csv.
(GH-30981)
https://github.com/python/cpython/commit/44afdbd5af4503e376148e9404b9c7a4f595b1fe
Change by Julien Palard :
--
pull_requests: +29160
pull_request: https://github.com/python/cpython/pull/30981
___
Python tracker
<https://bugs.python.org/issue42
Julien Palard added the comment:
I slowly started to work on rstlint in its own tree because working in a single
file felt inconfortable.
The more I advanced in this direction the more it made sense to me:
- I first added tests, I may have been able to do so in the cpython tree though
Change by Julien Palard :
--
pull_requests: +29059
pull_request: https://github.com/python/cpython/pull/30879
___
Python tracker
<https://bugs.python.org/issue42
Julien Palard added the comment:
Probably related to https://bugs.python.org/issue44011 and
https://github.com/MagicStack/uvloop/pull/385
--
___
Python tracker
<https://bugs.python.org/issue46
Julien Palard added the comment:
> dev docs direct to `/license.html` which redirects to `/3/license.html`
All docs are redirecting to `/license.html`, this allow it to work also out of
docs.python.org.
> 3.9 docs have the same; wouldn’t it be better to have `/3.9/license.html`?
It's
Julien Palard added the comment:
I don't know if it's related but _SSLPipe._shutdown_cb looks never called, in:
self._sslobj.unwrap()
self._sslobj = None
self._state = _UNWRAPPED
if self._shutdown_cb:
self._shutdown_cb()
the unwrap() call seems to always raise
Julien Palard added the comment:
I checked on docs.python.org and it's fixed.
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Julien Palard :
--
keywords: +patch
pull_requests: +28728
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/30527
___
Python tracker
<https://bugs.python.org/issu
Julien Palard added the comment:
This should be fixed in python-docs-theme==2022.1.
I'll close the issue when I actually see the fix applied on docs.python.org.
--
nosy: +mdk
___
Python tracker
<https://bugs.python.org/issue45
Julien Palard added the comment:
Feel like SSLProtocol's connection_lost should "bubble up" the info by calling
SSLProtocolTransport's close.
But I'm no familiar with this stack.
--
___
Python tracker
<https://bugs.python.o
Julien Palard added the comment:
Related to: https://bugs.python.org/issue23243
--
___
Python tracker
<https://bugs.python.org/issue46318>
___
___
Python-bug
New submission from Julien Palard :
Originally opened at: https://github.com/aio-libs/aiohttp/issues/6071
Reproducer:
import asyncio
class DumbProtocol(asyncio.Protocol):
def connection_made(self, transport):
print("Connection made")
self
Julien Palard added the comment:
Another true positive for make suspicious:
WARNING: [c-api/apiabiversion:70] ":macro" found in "the same format as the
c:macro:"
I'm working on implementing it on rstlint, in the meantime I opened:
=> https://github.com/python/cpy
Julien Palard added the comment:
A new false positive in b48ac6fe38b2fca9963b097c04cdecfc6083104e:
+++ b/Misc/NEWS.d/next/Tests/2021-11-23-12-36-21.bpo-45878.eOs_Mp.rst
@@ -0,0 +1,2 @@
+Test ``Lib/ctypes/test/test_functions.py::test_mro`` now uses
+``self.assertRaises
Julien Palard added the comment:
/!\ rstlint did triggered some false positive:
https://github.com/python/cpython/pull/29389/commits/f8ea20a0b51711b1c352c4aa4663cb953e7ebdc8
--
___
Python tracker
<https://bugs.python.org/issue42
Change by Julien Palard :
--
pull_requests: +27893
pull_request: https://github.com/python/cpython/pull/29652
___
Python tracker
<https://bugs.python.org/issue42
Change by Julien Palard :
--
pull_requests: +27867
pull_request: https://github.com/python/cpython/pull/29636
___
Python tracker
<https://bugs.python.org/issue42
Julien Palard added the comment:
A new false positive today from 31b3a70edb1216bdc8fab3b2eafd8ddb00487f41:
https://docs.python.org/dev/reference/datamodel.html#class-getitem-versus-getitem
in:
should be called::
from inspect import isclass
def subscribe(obj, x
New submission from Julien Palard :
Currently if ensurepip fails for a reason or another, for example if a wrong
module gets imported:
$ touch http.py
$ python3.10 -m venv .venv
Error: Command '['/tmp/.venv/bin/python3.10', '-Im', 'ensurepip', '--upgrade',
'--default-pip']' returned non-zero
New submission from Julien Palard :
In Doc/extending/newtypes one can read:
> This function is called with two Python objects and the operator as
> arguments, where the operator is one of Py_EQ, Py_NE, Py_LE, Py_GT, Py_LT or
> Py_GT.
It bet one of them should be Py_GE.
(I let
New submission from Julien Palard :
When running `python -c 'raise ValueError'` Python tries to open the ''
file, so:
$ echo Coucou > ''
$ python -c 'raise ValueError'
Traceback (most recent call last):
File "", line 1, in
Coucou
ValueError
--
New submission from Julien Palard :
I realized that many students get surprised by `python -m venv .venv` not
printing anything, a few even think it completly failed.
I'm OK teaching them this is normal, as `mv`, `cp`, `rm`, `django-admin
startproject`, ... does not print neither when
New submission from Julien Palard :
In IDLE, when hitting Ctrl-space on `matc` or `cas`, there's no completion to
`match` and `case.
References:
https://mail.python.org/archives/list/d...@python.org/thread/DRZIHLQU25JUD7IQKCOIIKI4ADB746MA/
--
assignee: terry.reedy
components: IDLE
Julien Palard added the comment:
DING DING DING DING DING DING, a new one today, and a true positive for `make
suspicious`, fixed in:
https://github.com/python/cpython/pull/28988/files
(along with the test added to rstlint)
--
___
Python tracker
Julien Palard added the comment:
New changeset d74da9e140441135a4eddaef9a37f00f32579038 by Josephine-Marie in
branch 'main':
bpo-45411: Update mimetypes.py (GH-28792)
https://github.com/python/cpython/commit/d74da9e140441135a4eddaef9a37f00f32579038
Julien Palard added the comment:
New suspicious today:
[library/typing:1011] "`" found in "# Type of ``val`` is narrowed
to ``list[str]``."
But it's only because the old one with List[str] instead of list[str] was in
susp-ignored.csv, so I just fixed susp-i
Change by Julien Palard :
--
pull_requests: +27143
pull_request: https://github.com/python/cpython/pull/28827
___
Python tracker
<https://bugs.python.org/issue42
Julien Palard added the comment:
New changeset c91b6f57f3f75b482e4a9d30ad2afe37892a8ceb by Julien Palard in
branch 'main':
bpo-10716: Migrating pydoc to html5. (GH-28651)
https://github.com/python/cpython/commit/c91b6f57f3f75b482e4a9d30ad2afe37892a8ceb
Julien Palard added the comment:
Those two looks legit to me.
--
nosy: +mdk
___
Python tracker
<https://bugs.python.org/issue45411>
___
___
Python-bugs-list m
Julien Palard added the comment:
Does someone want to review https://github.com/python/cpython/pull/28651?
--
___
Python tracker
<https://bugs.python.org/issue10
Julien Palard added the comment:
New changeset 9be930f9b169fb3d92693670ae069df902709b83 by Bibo-Joshi in branch
'main':
bpo-28206: Document signals Handlers, Sigmasks and Signals enums (GH-28628)
https://github.com/python/cpython/commit/9be930f9b169fb3d92693670ae069df902709b83
Julien Palard added the comment:
This still happen, but it's hard to tell if it'll happen from blurb, as when
it's two paragraphs consisting of a text and a list it does not happen, sphinx
just put the list as a sublist and it's clean.
I don't want to tell people stop using list, it's very
Julien Palard added the comment:
Tried myself at it.
I'm not a front-end guy, fasten your seatbelts.
I tried to keep it minimal, but after some work the change is not that small, I
focused on the HTML modernization only, not the design (to try to land at least
a first step toward
Change by Julien Palard :
--
pull_requests: +27019
pull_request: https://github.com/python/cpython/pull/28651
___
Python tracker
<https://bugs.python.org/issue10
Julien Palard added the comment:
New changeset 37272f5800ee1e9fcb2da4a1766366519b9b3d94 by Julien Palard in
branch 'main':
bpo-42238: [doc] remove unused, and deduplicate, suspicious ignore rules.
(GH-28137)
https://github.com/python/cpython/commit/37272f5800ee1e9fcb2da4a1766366519b9b3d94
Change by Julien Palard :
--
pull_requests: +26576
pull_request: https://github.com/python/cpython/pull/28137
___
Python tracker
<https://bugs.python.org/issue42
Julien Palard added the comment:
New changeset 21fa8547921d28b80b8a88d034081cab000d5480 by Julien Palard in
branch 'main':
bpo-42238: [doc] Some more make suspicious false positives. (GH-27945)
https://github.com/python/cpython/commit/21fa8547921d28b80b8a88d034081cab000d5480
Julien Palard added the comment:
10 false positives today, see:
https://github.com/python/cpython/pull/27945/files
I think I now need to put a deadline on this issue, let's say: one year without
true positives and I drop it, so 2021-07-19 for now.
(In the meantime, I continue to convert
Change by Julien Palard :
--
pull_requests: +26391
pull_request: https://github.com/python/cpython/pull/27945
___
Python tracker
<https://bugs.python.org/issue42
Julien Palard added the comment:
Another one \o/
Fix in: https://github.com/python/cpython/pull/27238
It is:
:func:pdb.main
Detected by make suspicious as:
WARNING: [whatsnew/changelog:320] ":func" found in ": Refactor argument
processing in :func:pdb.m
Change by Julien Palard :
--
pull_requests: +25787
pull_request: https://github.com/python/cpython/pull/27238
___
Python tracker
<https://bugs.python.org/issue42
Julien Palard added the comment:
New changeset 01331f1a3cf86fd308e9a134bb867bf01fb191f5 by Julien Palard in
branch 'main':
bpo-42238: rstlint: Add two new checks. (GH-26966)
https://github.com/python/cpython/commit/01331f1a3cf86fd308e9a134bb867bf01fb191f5
Change by Julien Palard :
--
pull_requests: +25530
pull_request: https://github.com/python/cpython/pull/26966
___
Python tracker
<https://bugs.python.org/issue42
Julien Palard added the comment:
Another true positive:
... versionchanged:: 3.11
from : 6cb145d23f5cf69b6d7414877d142747cd3d134c /
https://github.com/python/cpython/pull/26820
I also think it can be implemented in rstlint.
--
___
Python
Julien Palard added the comment:
> My computer is only configured for English. Running Firefox 90.0b9 (64-bit)
> on macOS 11.4 (20F71).
To add to this, it had nothing to do with your setup, the file was just lying
here on the server due to the previous build happening on the same hie
Julien Palard added the comment:
Thanks all for reporting and following on the issue.
The fix on docsbuild script worked as expected and the page now 404 as expected
as it does no logner exists in 3.11:
https://docs.python.org/3.11/library/parser.html
--
resolution: -> fixed
st
Julien Palard added the comment:
The previous one could probably be implemented in rstlint using (an equivalent
of):
git grep ':[a-z]\+:``[^:` ]+``' Doc/
Maybe specialized to known roles, like the script specializes to known
directives
Julien Palard added the comment:
Spotted a true positive in b19f45533942e4ad7ddf9d2d94f8b87c6f746bce:
:const:``None``
(I'm trying to build a true positive list, to have usefull cases where
suspicious is usefull, so in the long term I can maybe implement those cases in
rslint instead
Julien Palard added the comment:
I added a git clean in docsbuild scripts to switch from one version to another:
https://github.com/python/docsbuild-scripts/commit/1397a8dbe4c73744757ad24764baeb393842f30b
It should be enough to correctly start fresh from one lang to another and from
one
Julien Palard added the comment:
Hi all! Thanks for reporting! I'm taking a look at it.
--
assignee: docs@python -> mdk
___
Python tracker
<https://bugs.python.org/issu
Julien Palard added the comment:
New changeset 227a09325e7bf82ecd303b4696c054a086b29a00 by Julien Palard in
branch 'main':
bpo-42238: Doc CI: Disable suspicious checks. (GH-26575)
https://github.com/python/cpython/commit/227a09325e7bf82ecd303b4696c054a086b29a00
Change by Julien Palard :
--
pull_requests: +25163
pull_request: https://github.com/python/cpython/pull/26575
___
Python tracker
<https://bugs.python.org/issue42
Change by Julien Palard :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Julien Palard :
--
keywords: +easy, newcomer friendly
___
Python tracker
<https://bugs.python.org/issue43504>
___
___
Python-bugs-list mailin
Julien Palard added the comment:
I `rm -fr 3.10` on the doc server yesterday, so it started 3.10 from scratch
today and the file were properly removed.
--
___
Python tracker
<https://bugs.python.org/issue44
Julien Palard added the comment:
After the cron passed, and after purging the HTTP cache, it now 404:
$ curl https://docs.python.org/3.10/library/symbol.html
404 Not Found
Thanks for noticing!
--
resolution: -> fixed
stage: -> resolved
status: open -&g
Julien Palard added the comment:
This is interesting!
Also notice the footer: Last updated on Jul 26, 2020.
I checked on the docsbuild server and found the `symbol.html` file was marked
as modified today (probably by the build cron) (and it contains the "Last
updated on Jul 26,
Julien Palard added the comment:
Thanks Mohamed for the issue and Terry for fixing it.
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Julien Palard added the comment:
As far as this issue is concerned, it's fixed.
The discussion about this feature not working can continue in the appropriate
issue (23882)
--
resolution: -> fixed
stage: patch review -> resolved
status: open -&g
Julien Palard added the comment:
New changeset 2abbd8f2add5e80b86a965625b9a77ae94a101cd by Zackery Spytz in
branch 'master':
bpo-43954: Fix a missing word in the unittest docs (GH-25672)
https://github.com/python/cpython/commit/2abbd8f2add5e80b86a965625b9a77ae94a101cd
Julien Palard added the comment:
> I propose to remove the namespace package support entirely.
I'm no unittest expert, but I have nothing against removing it.
I totally agree to at least remove it from the doc while it does not w
Julien Palard added the comment:
I should have monitored this more closely, I started monitoring it weekly, then
life got over until today when I'm even surprised to see activity on the issue,
sry!
(Surprise leading me to investigate why I had not received notifications from
bpo, leading
New submission from Julien Palard :
In https://docs.python.org/3/library/unittest.html I see:
> Note that you need to the top level directory too.
Trying to guess, the missing word may be "specify":
> Changed in version 3.4: Test discovery supports namespace packages for st
Change by Julien Palard :
--
keywords: +patch
pull_requests: +23704
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/24945
___
Python tracker
<https://bugs.python.org/issu
New submission from Julien Palard :
In case an `.. audit-event::` has a content, Sphinx gets confused:
It will provide both "auto-generated" and the content in po files, for
interactivehook for example we have:
#: library/sys.rst:953
msgid ""
"Raise
New submission from Julien Palard :
effbot.org is down, it's currently displaying:
> effbot.org on hiatus
>
> effbot.org is taking a break. We’ll be back, in some form or another.
But docs.python.org have a few links pointing to it, `git grep effbot` finds 11
of them in the Doc/.
Change by Julien Palard :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Julien Palard added the comment:
New changeset 0269ce87c9347542c54a653dd78b9f60bb9fa822 by Chavdar Yotov in
branch 'master':
bpo-41933: Clarify wording for s * n in Common Sequence Operations (GH-22570)
https://github.com/python/cpython/commit/0269ce87c9347542c54a653dd78b9f60bb9fa822
Julien Palard added the comment:
I'm +1 on adding ** to match.
My first bet would be to add it to match, not adding a new method, nor a flag,
as it should not break compatibility:
It would only break iif someone have a `**` in a match AND does *not* expect it
to be recursive (as it would
Julien Palard added the comment:
I can reproduce it in 3.10 too.
--
versions: +Python 3.10
___
Python tracker
<https://bugs.python.org/issue43136>
___
___
Pytho
Change by Julien Palard :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Julien Palard added the comment:
New changeset eb9983c59b0683270328b5c40a115bb028209511 by Julien Palard in
branch 'master':
bpo-42869: Avoid an HTTP redirection. (GH-24174)
https://github.com/python/cpython/commit/eb9983c59b0683270328b5c40a115bb028209511
Julien Palard added the comment:
Due to https://github.com/python/cpython/pull/24282 this is sadly un-fixed.
Either we find another way to fix this, either we wait 3 releases and we
re-commit https://github.com/python/cpython/pull/23620.
--
resolution: fixed -> later
stage: resol
Change by Julien Palard :
--
resolution: -> not a bug
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Julien Palard added the comment:
New changeset 5c1f15b4b1024cbf0acc85832f0c623d1a4605fd by Julien Palard in
branch 'master':
bpo-42843: Keep Sphinx 1.8 and Sphinx 2 compatibility (GH-24282)
https://github.com/python/cpython/commit/5c1f15b4b1024cbf0acc85832f0c623d1a4605fd
Julien Palard added the comment:
Nice find! I am able to reproduce it too in many Python releases.
I see differnt ways we can fix it:
# Using a random secret generated at startup time
Used any way, like by providing an hmac on getfile urls to ensure they are
signed with the server secret
Change by Julien Palard :
--
pull_requests: +23103
pull_request: https://github.com/python/cpython/pull/24282
___
Python tracker
<https://bugs.python.org/issue42
Julien Palard added the comment:
Question: Why do we keep setuptools?
According to PEP453:
> Once pip is able to run pip install --upgrade pip without needing setuptools
> installed first, then the private copy of setuptools will be removed from
> ensurepip in subsequent CPython
Julien Palard added the comment:
This is now fixed on the docs server thanks to [1], so it should work for
already release Python 3.9 :)
[1]: https://github.com/python/psf-salt/pull/198
I still did a PR on cpython to avoid a redirection, and get cpython itself out
of the "peopl
1 - 100 of 685 matches
Mail list logo