Re: svnpubsub/commit-hook.py:65 - variable url assigned but not used

2021-05-14 Thread Daniel Shahaf
sebb wrote on Fri, 14 May 2021 11:44 +00:00: > On Fri, 14 May 2021 at 12:22, Daniel Shahaf wrote: > > sebb wrote on Thu, 13 May 2021 14:29 +00:00: > > > > Is it misspelt? > > > > ITYM "misspelled" ;-) > > No, I am using British English. I know. That's what the emoticon was there to

Re: svnpubsub/commit-hook.py:65 - variable url assigned but not used

2021-05-14 Thread sebb
On Fri, 14 May 2021 at 12:22, Daniel Shahaf wrote: > > sebb wrote on Thu, 13 May 2021 14:29 +00:00: > > On Thu, 13 May 2021 at 15:16, sebb wrote: > > > On Thu, 13 May 2021 at 15:11, Daniel Shahaf > > > wrote: > > > > > > > > sebb wrote on Wed, May 12, 2021 at 11:49:41 +0100: > > > > > As the

Re: svnpubsub/commit-hook.py:65 - variable url assigned but not used

2021-05-14 Thread Daniel Shahaf
sebb wrote on Thu, 13 May 2021 14:29 +00:00: > On Thu, 13 May 2021 at 15:16, sebb wrote: > > On Thu, 13 May 2021 at 15:11, Daniel Shahaf wrote: > > > > > > sebb wrote on Wed, May 12, 2021 at 11:49:41 +0100: > > > > As the subject says > > > > > > Assuming opener.open() actually returns a URL, I

Re: svnpubsub/commit-hook.py:65 - variable url assigned but not used

2021-05-13 Thread sebb
Furthermore, the method does not actually return a URL. The method behaves like the following: https://python.readthedocs.io/en/v2.7.2/library/urllib2.html#urllib2.urlopen On Thu, 13 May 2021 at 15:16, sebb wrote: > > The problem is that it is not clear whether the variable is supposed > to be

Re: svnpubsub/commit-hook.py:65 - variable url assigned but not used

2021-05-13 Thread sebb
The problem is that it is not clear whether the variable is supposed to be used or not. Is it misspelt? AIUI the convention for intentionally unused variables is to prefix the name with an underscore, as this no longer triggers the warning in PyLint On Thu, 13 May 2021 at 15:11, Daniel Shahaf

Re: svnpubsub/commit-hook.py:65 - variable url assigned but not used

2021-05-13 Thread Daniel Shahaf
sebb wrote on Wed, May 12, 2021 at 11:49:41 +0100: > As the subject says Assuming opener.open() actually returns a URL, I don't see the problem here. The variable documents the return type for anyone who may want to extend the function.

svnpubsub/commit-hook.py:65 - variable url assigned but not used

2021-05-12 Thread sebb
As the subject says