Re: [Django] #27463: ambiguous variable name 'l'

[Django]

#27463: ambiguous variable name 'l'
-+-
 Reporter:  Ramin Farajpour  |Owner:  Ramin
  Cami   |  Farajpour Cami
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  Core (Other) |  Version:  1.10
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:
 |  Unreviewed
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Aymeric Augustin):

 I have yet to come across a monospace font where `l` and `I` are
 identical, but it takes less effort to accept a PR making that change than
 to discuss.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.5ebc388855ae897ce8c96dd1f65fc42a%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27463: ambiguous variable name 'l'

[Django]

#27463: ambiguous variable name 'l'
-+-
 Reporter:  Ramin Farajpour  |Owner:  Ramin
  Cami   |  Farajpour Cami
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  Core (Other) |  Version:  1.10
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:
 |  Unreviewed
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Ramin Farajpour Cami):

 pycodestyle member tell me :
 https://github.com/PyCQA/pycodestyle/issues/598

 Allow me for PR ?

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.cdacc9111070c84c29236f722486e37a%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27463: ambiguous variable name 'l'

[Django]

#27463: ambiguous variable name 'l'
-+-
 Reporter:  Ramin Farajpour  |Owner:  Ramin
  Cami   |  Farajpour Cami
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  Core (Other) |  Version:  1.10
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:
 |  Unreviewed
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Ramin Farajpour Cami):

 really!

 Flake8 uses ranges for mccabe, pyflakes, and pycodestyle because each of
 those projects tend to add new checks in minor releases.

 http://flake8.pycqa.org/en/latest/faq.html#why-does-flake8-use-ranges-for-
 its-dependencies

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.abf71f3bf928807d371c508cdd251cd4%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27463: ambiguous variable name 'l'

[Django]

#27463: ambiguous variable name 'l'
-+-
 Reporter:  Ramin Farajpour  |Owner:  Ramin
  Cami   |  Farajpour Cami
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  Core (Other) |  Version:  1.10
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:
 |  Unreviewed
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Tim Graham):

 * component:  Utilities => Core (Other)


Comment:

 I'm not sure about this, particularly if our style tool (`flake8`) doesn't
 provide automated enforcement. A font that doesn't distinguish between one
 character or another doesn't seem so useful for programmers.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.d43d088a8321ac72b706862d926b9970%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27463: ambiguous variable name 'l'

[Django]

#27463: ambiguous variable name 'l'
-+-
 Reporter:  Ramin Farajpour  |Owner:  Ramin
  Cami   |  Farajpour Cami
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  Utilities|  Version:  1.10
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:
 |  Unreviewed
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Ramin Farajpour Cami):

 https://pep8.readthedocs.io/en/latest/_modules/pycodestyle.html

 {{{
 Okay: L = 0
 Okay: o = 123
 Okay: i = 42
 E741: l = 0
 E741: O = 123
 E741: I = 42
 }}}

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.43073f4fe1848870b73f2564b027bcd0%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27463: ambiguous variable name 'l'

[Django]

#27463: ambiguous variable name 'l'
-+-
 Reporter:  Ramin Farajpour  |Owner:  Ramin
  Cami   |  Farajpour Cami
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  Utilities|  Version:  1.10
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:
 |  Unreviewed
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Ramin Farajpour Cami):

 * owner:  nobody => Ramin Farajpour Cami
 * status:  new => assigned


--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.4377c16ee461a78ca38a0bdfe9c23581%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

[Django] #27463: ambiguous variable name 'l'

[Django]

#27463: ambiguous variable name 'l'
+
   Reporter:  Ramin Farajpour Cami  |  Owner:  nobody
   Type:  Cleanup/optimization  | Status:  new
  Component:  Utilities |Version:  1.10
   Severity:  Normal|   Keywords:
   Triage Stage:  Unreviewed|  Has patch:  0
Needs documentation:  0 |Needs tests:  0
Patch needs improvement:  0 |  Easy pickings:  0
  UI/UX:  0 |
+
 Hi,

 In some fonts, character 'l' are indistinguishable from the numeral one.
 So better use other name instead.

 https://github.com/django/django/blob/master/tests/queries/tests.py#L567&L574

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/050.1dfe52554e7be8ed04ba415b21963cce%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.