Python: Remove references to LGTM

Author: calumgrant

python/ql/test/3/library-tests/PointsTo/typehints/test.py

@@ -8,11 +8,3 @@ def bar(s:set)->Set:
 
 t1 = Optional[Optional[int]]
 t2 = Optional[int][int]
-
-# ODASA-8075
-# Commented out until the fix has been pushed to LGTM.com
-#class baz():
-#    pass
-#
-#while True:
-#    baz = baz[baz]

python/ql/test/library-tests/regex/charSetTest.py

@@ -25,14 +25,12 @@
 
 re.compile("]]][[[[]") #$ charSet=3:8
 
-
-#ODASA-3985
 #Half Surrogate pairs
 re.compile(u'[\uD800-\uDBFF][\uDC00-\uDFFF]') #$ charSet=0:5 charSet=5:10
 #Outside BMP
 re.compile(u'[\U00010000-\U0010ffff]') #$ charSet=0:5
 
-#Misparsed on LGTM
+#Misparsed
 re.compile(r"\[(?P<txt>[^[]*)\]\((?P<uri>[^)]*)") #$ charSet=10:14 charSet=28:32
 
  # parses wrongly, sees this   \|/ as a char set start

python/ql/test/library-tests/regex/escapedCharacterTest.py

@@ -21,7 +21,7 @@
 #Outside BMP
 re.compile(u'[\U00010000-\U0010ffff]') # not escapes
 
-#Misparsed on LGTM
+#Misparsed
 re.compile(r"\[(?P<txt>[^[]*)\]\((?P<uri>[^)]*)") #$ escapedCharacter=0:2 escapedCharacter=16:18 escapedCharacter=18:20
 
 #Non-raw string

python/ql/test/library-tests/regex/test.py

@@ -62,7 +62,7 @@
 #Named group with caret and empty choice.
 re.compile(r'(?:(?P<n1>^(?:|x)))')
 
-#Misparsed on LGTM
+#Misparsed
 re.compile(r"\[(?P<txt>[^[]*)\]\((?P<uri>[^)]*)")
 
 re.compile("", re.M) # ODASA-8056

python/ql/test/library-tests/regexparser/unittests.py

@@ -11,7 +11,7 @@
 # minimal example constructed by @erik-krogh
 baz = re.compile(r'\+0')
 
-# exerpts from LGTM.com
+# exerpts from real-world results
 re.compile(r'\+0x')
 re.compile(r'\+0x.*')
 re.compile(r'+\-0+\.')

python/ql/test/query-tests/Classes/conflicting/odasa6643.py

@@ -1,8 +1,6 @@
 #This code has conflicting attributes,
 #but the documentation in the standard library tells you do it this way :(
 
-#See https://discuss.lgtm.com/t/warning-on-normal-use-of-python-socketserver-mixins/677
-
 class ThreadingMixIn(object):
 
     def process_request(selfself, req):

python/ql/test/query-tests/Classes/missing-init/missing_init.py

@@ -39,7 +39,7 @@ class UVT(UT, PU):
 class IUVT(IUT, UVT):
     pass
 
-#False positive observed on LGTM
+#False positive
 class M1(object):
     def __init__(self):
         print("A")

python/ql/test/query-tests/Expressions/eq/expressions_test.py

@@ -78,7 +78,7 @@ def __eq__(self, other):
         '''
         return self is other
 
-#Was FP -- https://github.com/lgtmhq/lgtm-queries/issues/13
+#Was FP
 def both_sides_known(zero_based="auto", query_id=False):
     if query_id and zero_based == "auto":
         zero_based = True

python/ql/test/query-tests/Functions/general/explicit_return_in_init.py

@@ -46,7 +46,7 @@ def __init__(self):
 
 # OK as procedure implicitly returns None
 #
-# this was seen in the wild: https://lgtm.com/projects/b/jjburton/cgmtools/snapshot/0d8a429b7ea17854a5e5341df98b1cbd54d7fe6c/files/mayaTools/cgm/lib/classes/AttrFactory.py?sort=name&dir=ASC&mode=heatmap#L90
+# this was seen in the wild:
 # using a pattern of `return procedure_that_logs_error()`
 
 def procedure():

python/ql/test/query-tests/Functions/general/parameter_names.py

@@ -127,7 +127,7 @@ def meth(arg):
 # makes it such. As a consequence, the query `py/not-named-self` and other
 # relevant queries need to account for this.
 #
-# This has come up in the wild via LGTM as a false positive. For example,
+# This has come up in the wild as a false positive. For example,
 # `__init_subclass__`:
 #   https://docs.python.org/3/reference/datamodel.html#customizing-class-creation
 # `__class_getitem__`: