Re: svnpubsub/commit-hook.py:65 - variable url assigned but not used
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 acknowledge.
Re: svnpubsub/commit-hook.py:65 - variable url assigned but not used
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 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. > > > > > > The problem is that it is not clear whether the variable is supposed > > > to be used or not. > > Then add a comment, or return the variable, etc.. Or drop the variable entirely. > > > Is it misspelt? > > ITYM "misspelled" ;-) No, I am using British English. > > > AIUI the convention for intentionally unused variables is to prefix > > > the name with an underscore, as this no longer triggers the warning in > > > PyLint > > Last I checked, docs.python.org said (single) leading underscores designated > class members that weren't part of the class's public API. > > > Furthermore, the method does not actually return a URL. > > Then the variable could be renamed. Or dropped. > > The method behaves like the following: > > https://python.readthedocs.io/en/v2.7.2/library/urllib2.html#urllib2.urlopen > > Thanks. > > Also, all the above discussion applies to revprop-change-hook.py, too. > > Daniel
Re: svnpubsub/commit-hook.py:65 - variable url assigned but not used
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 don't see the problem > > > here. The variable documents the return type for anyone who may want > > > to extend the function. > > > > The problem is that it is not clear whether the variable is supposed > > to be used or not. Then add a comment, or return the variable, etc.. > > Is it misspelt? ITYM "misspelled" ;-) > > AIUI the convention for intentionally unused variables is to prefix > > the name with an underscore, as this no longer triggers the warning in > > PyLint Last I checked, docs.python.org said (single) leading underscores designated class members that weren't part of the class's public API. > Furthermore, the method does not actually return a URL. Then the variable could be renamed. > The method behaves like the following: > https://python.readthedocs.io/en/v2.7.2/library/urllib2.html#urllib2.urlopen Thanks. Also, all the above discussion applies to revprop-change-hook.py, too. Daniel
Re: svnpubsub/commit-hook.py:65 - variable url assigned but not used
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 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 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 don't see the problem > > here. The variable documents the return type for anyone who may want > > to extend the function.
Re: svnpubsub/commit-hook.py:65 - variable url assigned but not used
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 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 don't see the problem > here. The variable documents the return type for anyone who may want > to extend the function.
Re: svnpubsub/commit-hook.py:65 - variable url assigned but not used
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
As the subject says