Wed, Feb 21
Tue, Feb 20
Sun, Feb 18
I am not familar with toolforge. For pywikibot requests >= 2.9 is required to solve T184713 and will be mandatory after https://gerrit.wikimedia.org/r/#/c/406579/ will be deployed. Anyway this restriction is only necessary when using sseclient for eventstreams (and btw. sseclient installation must be 0.0.18 or higher). If this could be solved by users themself I am fine with it but I fear not all are experienced enought to set it up in a needed way. So either there should be a way preinstalling requests >= 2.9 or any help page for users how to proceed in this matter.
I reverted this patch due to to many problems shown by tests
I think this patch doesn't care current installations. Currently requests>=2.4.1 is required I guess for socket_timeout. If an older release is installed, config2.py patches it and the pwb is still working. Now we require 2.9 which is needed for sseclient. If an older release is installed all parts are still working. With this pathc EventStreams which gives an error message to update the requests version. Without this patch no warning will be shown but EventStreams doesn't work anyway.
See also T166539, hm.
I am wondering: in current requests-requirements.txt and requirements.txt the entries are
Why does the default version care then?
Sat, Feb 17
The Response indicates the problem: Http response status 503, see https://de.wikipedia.org/wiki/HTTP-Statuscode#5xx_–_Server-Fehler, the server may be down
Wed, Feb 14
Tue, Feb 13
I agree with Dalba. I've no idea for what the major release stands except 1.0 means trunk, 2.0 means the interim rewrite stable branch and 3 the current master which never will change in future except we have a complete code redesign.
Mon, Feb 12
Is this doc clone really needed?
Increasing minor version for backward incompatibility? Is there any package that does that? I think it'll confuse users.
Sun, Feb 11
Commonscat template is already inluded as default.
Mon, Feb 5
Sun, Feb 4
Sat, Feb 3
Sure this is not a regression against compat but a difference which should be noted at compat2core.py and mayby other instructions too.
There are more than one possibility to specify a generator for this script. I think this warning is good enough to ask for it.
Patch reverted with https://gerrit.wikimedia.org/r/#/c/407889/
Fri, Feb 2
@Dalba: It is required to use EventStreams after the last change on server side. I checked every release of requests and sseclient to find the right combination. Anyway it the requests < 2.9 the framework can be used except of EventStreams which either finds empty messages or runs into a loop inside requests without any response. Maybe this should be solved upstreams then.
This issue is resolved prior by disabling E402 and removing version strings which was the most discussion part. This might be enough for now.
Thu, Feb 1
Sure but urls are ignored by that rule for good reasons.
@Dvorava: let’s have a Monitor with 1920 dots resolution so see patch diffs side by side in a normal char size of 12 points. Gives 80 chars. Ok say 10 points is enough. But you need some frames an line numbers. I think this is a good reason for 80 chars limit. Or take python‘s IDLE. The default wide is 80 chars. Good enough to pin to windows side by side without needing a slider so show the lines. You may use brackets zu continue lines too, divide code in smaller parts to avoid indenting, shorten expressions by assigning it to identifiers etc.