Adam Williamson

Results 433 comments of Adam Williamson

OK, so with new astroid and dill and #7153 , here's some really-python3.11 issues: * telnetlib [is deprecated in 3.11](https://docs.python.org/3.11/whatsnew/3.11.html#deprecated), that breaks `access_attr_before_def_false_positive.py` * `asyncio.coroutine` was [removed in 3.11](https://docs.python.org/3.11/whatsnew/3.11.html#removed), that...

Hmm, this is a fun one. the pylint output for the `syntax_error.py` test has changed, which breaks both that test and `import_error` (which imports it). It changed from just "invalid...

Thanks. I'm trying to work on fixes for the things I'm sure are python 3.11 (and not just general running-the-tests-downstream stuff, or already-fixed-since-2.14.4 stuff), but I don't know the codebase...

Thanks! The fork is https://github.com/AdamWill/pylint/commits/py311 , with my attempts at fixes for the telnetlib and asyncio.coroutine issues so far. I'll send a PR when I'm done looking through the other...

@DanielNoord could we say that `test_stdin_syntaxerror` in `tests/test_self.py` already covers the same ground as the syntaxerror functional test? edit: hmm, but that doesn't account for `import_error`'s use of it. mmf.

With the changes I've submitted so far, plus a downstream patch for those two specific cases where the syntax error text changed, the test suite fully passes. So I don't...

My problem with moving it to the unittests is then how do we keep the bit of `import-error` that tries to import `syntax-error.py` and expects it to throw specific errors?...

Was the `Enum.__init__` thing resolved somehow? My PR wasn't merged and the upstream comment hasn't had a reply yet..

We have the visual glitches with the menus fixed on Fedora, so you can use the dropdown menu there, but selecting another choice doesn't actually seem to *work*, as described...

FWIW, openQA schedules tests differently, and in a way that happens to handle this (mainly because we wrote the scheduling code before the `koji-build-group.build.complete` message existed). We schedule on the...