summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--doc/en-us/config.md15
-rw-r--r--doc/pt-br/config.md15
-rw-r--r--doc/zh-cn/config.md15
-rw-r--r--doc/zh-tw/config.md15
-rw-r--r--locale/en-us/script.lua5
-rw-r--r--locale/en-us/setting.lua6
-rw-r--r--locale/pt-br/script.lua4
-rw-r--r--locale/pt-br/setting.lua6
-rw-r--r--locale/zh-cn/script.lua4
-rw-r--r--locale/zh-cn/setting.lua6
-rw-r--r--locale/zh-tw/script.lua4
-rw-r--r--locale/zh-tw/setting.lua6
-rw-r--r--script/core/diagnostics/uppercase-local.lua47
-rw-r--r--script/proto/diagnostic.lua1
-rw-r--r--test/diagnostics/init.lua3
-rw-r--r--test/diagnostics/uppercase-local.lua66
16 files changed, 21 insertions, 197 deletions
diff --git a/doc/en-us/config.md b/doc/en-us/config.md
index 96a023b9..0e5d09ee 100644
--- a/doc/en-us/config.md
+++ b/doc/en-us/config.md
@@ -329,7 +329,6 @@ Array<string>
* ``"unused-label"``
* ``"unused-local"``
* ``"unused-vararg"``
-* ``"uppercase-local"``
## default
@@ -471,7 +470,6 @@ object<string, string>
"strict": "Fallback",
/*
* global-element
- * uppercase-local
*/
"strict-convention": "None",
/*
@@ -595,7 +593,6 @@ object<string, string>
"strict": "Fallback",
/*
* global-element
- * uppercase-local
*/
"strict-convention": "Fallback",
/*
@@ -929,11 +926,7 @@ object<string, string>
/*
Enable unused vararg diagnostics.
*/
- "unused-vararg": "Opened",
- /*
- Enable diagnostics to warn about local element names starting with an uppercase letter.
- */
- "uppercase-local": "None"
+ "unused-vararg": "Opened"
}
```
@@ -1185,11 +1178,7 @@ object<string, string>
/*
Enable unused vararg diagnostics.
*/
- "unused-vararg": "Hint",
- /*
- Enable diagnostics to warn about local element names starting with an uppercase letter.
- */
- "uppercase-local": "Warning"
+ "unused-vararg": "Hint"
}
```
diff --git a/doc/pt-br/config.md b/doc/pt-br/config.md
index 952980d8..762f2259 100644
--- a/doc/pt-br/config.md
+++ b/doc/pt-br/config.md
@@ -329,7 +329,6 @@ Array<string>
* ``"unused-label"``
* ``"unused-local"``
* ``"unused-vararg"``
-* ``"uppercase-local"``
## default
@@ -471,7 +470,6 @@ object<string, string>
"strict": "Fallback",
/*
* global-element
- * uppercase-local
*/
"strict-convention": "None",
/*
@@ -595,7 +593,6 @@ object<string, string>
"strict": "Fallback",
/*
* global-element
- * uppercase-local
*/
"strict-convention": "Fallback",
/*
@@ -929,11 +926,7 @@ object<string, string>
/*
未使用的不定参数
*/
- "unused-vararg": "Opened",
- /*
- Enable diagnostics to warn about local element names starting with an uppercase letter.
- */
- "uppercase-local": "None"
+ "unused-vararg": "Opened"
}
```
@@ -1185,11 +1178,7 @@ object<string, string>
/*
未使用的不定参数
*/
- "unused-vararg": "Hint",
- /*
- Enable diagnostics to warn about local element names starting with an uppercase letter.
- */
- "uppercase-local": "Warning"
+ "unused-vararg": "Hint"
}
```
diff --git a/doc/zh-cn/config.md b/doc/zh-cn/config.md
index 294fe046..869e2506 100644
--- a/doc/zh-cn/config.md
+++ b/doc/zh-cn/config.md
@@ -329,7 +329,6 @@ Array<string>
* ``"unused-label"``
* ``"unused-local"``
* ``"unused-vararg"``
-* ``"uppercase-local"``
## default
@@ -471,7 +470,6 @@ object<string, string>
"strict": "Fallback",
/*
* global-element
- * uppercase-local
*/
"strict-convention": "None",
/*
@@ -595,7 +593,6 @@ object<string, string>
"strict": "Fallback",
/*
* global-element
- * uppercase-local
*/
"strict-convention": "Fallback",
/*
@@ -929,11 +926,7 @@ object<string, string>
/*
未使用的不定参数
*/
- "unused-vararg": "Opened",
- /*
- Enable diagnostics to warn about local element names starting with an uppercase letter.
- */
- "uppercase-local": "None"
+ "unused-vararg": "Opened"
}
```
@@ -1184,11 +1177,7 @@ object<string, string>
/*
未使用的不定参数
*/
- "unused-vararg": "Hint",
- /*
- Enable diagnostics to warn about local element names starting with an uppercase letter.
- */
- "uppercase-local": "Warning"
+ "unused-vararg": "Hint"
}
```
diff --git a/doc/zh-tw/config.md b/doc/zh-tw/config.md
index e12bfde7..c922ad6b 100644
--- a/doc/zh-tw/config.md
+++ b/doc/zh-tw/config.md
@@ -329,7 +329,6 @@ Array<string>
* ``"unused-label"``
* ``"unused-local"``
* ``"unused-vararg"``
-* ``"uppercase-local"``
## default
@@ -471,7 +470,6 @@ object<string, string>
"strict": "Fallback",
/*
* global-element
- * uppercase-local
*/
"strict-convention": "None",
/*
@@ -595,7 +593,6 @@ object<string, string>
"strict": "Fallback",
/*
* global-element
- * uppercase-local
*/
"strict-convention": "Fallback",
/*
@@ -929,11 +926,7 @@ object<string, string>
/*
未使用的不定引數
*/
- "unused-vararg": "Opened",
- /*
- Enable diagnostics to warn about local element names starting with an uppercase letter.
- */
- "uppercase-local": "None"
+ "unused-vararg": "Opened"
}
```
@@ -1184,11 +1177,7 @@ object<string, string>
/*
未使用的不定引數
*/
- "unused-vararg": "Hint",
- /*
- Enable diagnostics to warn about local element names starting with an uppercase letter.
- */
- "uppercase-local": "Warning"
+ "unused-vararg": "Hint"
}
```
diff --git a/locale/en-us/script.lua b/locale/en-us/script.lua
index e7f4b98b..2a13214f 100644
--- a/locale/en-us/script.lua
+++ b/locale/en-us/script.lua
@@ -150,11 +150,8 @@ DIAG_INVISIBLE_PROTECTED =
'Field `{field}` is protected, it can only be accessed in class `{class}` and its subclasses.'
DIAG_INVISIBLE_PACKAGE =
'Field `{field}` can only be accessed in same file `{uri}`.'
-DIAG_GLOBAL_ELEMENT =
+DIAG_GLOBAL_ELEMENT =
'Element is global.'
-DIAG_UPPERCASE_LOCAL =
-'Local element `{}` must start with lowercase letter'
-
MWS_NOT_SUPPORT =
'{} does not support multi workspace for now, I may need to restart to support the new workspace ...'
diff --git a/locale/en-us/setting.lua b/locale/en-us/setting.lua
index b03e3e43..9f606cd7 100644
--- a/locale/en-us/setting.lua
+++ b/locale/en-us/setting.lua
@@ -415,10 +415,8 @@ config.diagnostics['unknown-operator'] =
'Enable diagnostics for unknown operators.'
config.diagnostics['unreachable-code'] =
'Enable diagnostics for unreachable code.'
-config.diagnostics['global-element'] =
-'Enable diagnostics to warn about global variables.'
-config.diagnostics['uppercase-local'] =
-'Enable diagnostics to warn about local element names starting with an uppercase letter.'
+config.diagnostics['global-element'] =
+'Enable diagnostics to warn about global elements.'
config.typeFormat.config =
'Configures the formatting behavior while typing Lua code.'
config.typeFormat.config.auto_complete_end =
diff --git a/locale/pt-br/script.lua b/locale/pt-br/script.lua
index 6497c520..affcd893 100644
--- a/locale/pt-br/script.lua
+++ b/locale/pt-br/script.lua
@@ -150,10 +150,8 @@ DIAG_INVISIBLE_PROTECTED = -- TODO: need translate!
'Field `{field}` is protected, it can only be accessed in class `{class}` and its subclasses.'
DIAG_INVISIBLE_PACKAGE = -- TODO: need translate!
'Field `{field}` can only be accessed in same file `{uri}`.'
-DIAG_GLOBAL_ELEMENT = -- TODO: need translate!
+DIAG_GLOBAL_ELEMENT = -- TODO: need translate!
'Element is global.'
-DIAG_UPPERCASE_LOCAL = -- TODO: need translate!
-'Local element `{}` must start with lowercase letter'
MWS_NOT_SUPPORT =
'{} não é suportado múltiplos espaços de trabalho por enquanto, posso precisar reiniciar para estabelecer um novo espaço de trabalho ...'
diff --git a/locale/pt-br/setting.lua b/locale/pt-br/setting.lua
index ec94f575..95f26b76 100644
--- a/locale/pt-br/setting.lua
+++ b/locale/pt-br/setting.lua
@@ -415,10 +415,8 @@ config.diagnostics['unknown-operator'] = -- TODO: need translate!
'Enable diagnostics for unknown operators.'
config.diagnostics['unreachable-code'] = -- TODO: need translate!
'Enable diagnostics for unreachable code.'
-config.diagnostics['global-element'] = -- TODO: need translate!
-'Enable diagnostics to warn about global variables.'
-config.diagnostics['uppercase-local'] = -- TODO: need translate!
-'Enable diagnostics to warn about local element names starting with an uppercase letter.'
+config.diagnostics['global-element'] = -- TODO: need translate!
+'Enable diagnostics to warn about global elements.'
config.typeFormat.config = -- TODO: need translate!
'Configures the formatting behavior while typing Lua code.'
config.typeFormat.config.auto_complete_end = -- TODO: need translate!
diff --git a/locale/zh-cn/script.lua b/locale/zh-cn/script.lua
index da817698..f6f639ea 100644
--- a/locale/zh-cn/script.lua
+++ b/locale/zh-cn/script.lua
@@ -150,10 +150,8 @@ DIAG_INVISIBLE_PROTECTED =
'字段 `{field}` 受到保护,只能在 `{class}` 类极其子类中才能访问。'
DIAG_INVISIBLE_PACKAGE =
'字段 `{field}` 只能在相同的文件 `{uri}` 中才能访问。'
-DIAG_GLOBAL_ELEMENT = -- TODO: need translate!
+DIAG_GLOBAL_ELEMENT = -- TODO: need translate!
'Element is global.'
-DIAG_UPPERCASE_LOCAL = -- TODO: need translate!
-'Local element `{}` must start with lowercase letter'
MWS_NOT_SUPPORT =
'{} 目前还不支持多工作目录,我可能需要重启才能支持新的工作目录...'
diff --git a/locale/zh-cn/setting.lua b/locale/zh-cn/setting.lua
index e1eb7de9..731190e2 100644
--- a/locale/zh-cn/setting.lua
+++ b/locale/zh-cn/setting.lua
@@ -414,10 +414,8 @@ config.diagnostics['unknown-operator'] = -- TODO: need translate!
'Enable diagnostics for unknown operators.'
config.diagnostics['unreachable-code'] = -- TODO: need translate!
'Enable diagnostics for unreachable code.'
-config.diagnostics['global-element'] = -- TODO: need translate!
-'Enable diagnostics to warn about global variables.'
-config.diagnostics['uppercase-local'] = -- TODO: need translate!
-'Enable diagnostics to warn about local element names starting with an uppercase letter.'
+config.diagnostics['global-element'] = -- TODO: need translate!
+'Enable diagnostics to warn about global elements.'
config.typeFormat.config = -- TODO: need translate!
'Configures the formatting behavior while typing Lua code.'
config.typeFormat.config.auto_complete_end = -- TODO: need translate!
diff --git a/locale/zh-tw/script.lua b/locale/zh-tw/script.lua
index 77ff6fea..4d9a89ad 100644
--- a/locale/zh-tw/script.lua
+++ b/locale/zh-tw/script.lua
@@ -150,10 +150,8 @@ DIAG_INVISIBLE_PROTECTED = -- TODO: need translate!
'Field `{field}` is protected, it can only be accessed in class `{class}` and its subclasses.'
DIAG_INVISIBLE_PACKAGE = -- TODO: need translate!
'Field `{field}` can only be accessed in same file `{uri}`.'
-DIAG_GLOBAL_ELEMENT = -- TODO: need translate!
+DIAG_GLOBAL_ELEMENT = -- TODO: need translate!
'Element is global.'
-DIAG_UPPERCASE_LOCAL = -- TODO: need translate!
-'Local element `{}` must start with lowercase letter'
MWS_NOT_SUPPORT =
'{} 目前還不支援多工作目錄,我可能需要重新啟動才能支援新的工作目錄...'
diff --git a/locale/zh-tw/setting.lua b/locale/zh-tw/setting.lua
index ebf615ac..fd363d3f 100644
--- a/locale/zh-tw/setting.lua
+++ b/locale/zh-tw/setting.lua
@@ -414,10 +414,8 @@ config.diagnostics['unknown-operator'] = -- TODO: need translate!
'Enable diagnostics for unknown operators.'
config.diagnostics['unreachable-code'] = -- TODO: need translate!
'Enable diagnostics for unreachable code.'
-config.diagnostics['global-element'] = -- TODO: need translate!
-'Enable diagnostics to warn about global variables.'
-config.diagnostics['uppercase-local'] = -- TODO: need translate!
-'Enable diagnostics to warn about local element names starting with an uppercase letter.'
+config.diagnostics['global-element'] = -- TODO: need translate!
+'Enable diagnostics to warn about global elements.'
config.typeFormat.config = -- TODO: need translate!
'Configures the formatting behavior while typing Lua code.'
config.typeFormat.config.auto_complete_end = -- TODO: need translate!
diff --git a/script/core/diagnostics/uppercase-local.lua b/script/core/diagnostics/uppercase-local.lua
deleted file mode 100644
index cc377dd2..00000000
--- a/script/core/diagnostics/uppercase-local.lua
+++ /dev/null
@@ -1,47 +0,0 @@
-local files = require 'files'
-local guide = require 'parser.guide'
-local lang = require 'language'
-
-local function isDocClass(source)
- if not source.bindDocs then
- return false
- end
- for _, doc in ipairs(source.bindDocs) do
- if doc.type == 'doc.class' then
- return true
- end
- end
- return false
-end
-
--- If local elements must be named lowercase by coding convention, this diagnostic helps with reminding about that
-return function (uri, callback)
- local ast = files.getState(uri)
- if not ast then
- return
- end
-
- guide.eachSourceType(ast.ast, 'local', function (source)
- local name = guide.getKeyName(source)
- if not name then
- return
- end
- local first = name:match '%w' -- alphanumeric character
- if not first then
- return
- end
- if not first:match '%u' then -- uppercase
- return
- end
- -- If the assignment is marked as doc.class, then it is considered allowed
- if isDocClass(source) then
- return
- end
-
- callback {
- start = source.start,
- finish = source.finish,
- message = lang.script('DIAG_UPPERCASE_LOCAL', name),
- }
- end)
-end
diff --git a/script/proto/diagnostic.lua b/script/proto/diagnostic.lua
index a6232b8a..bea7c160 100644
--- a/script/proto/diagnostic.lua
+++ b/script/proto/diagnostic.lua
@@ -173,7 +173,6 @@ m.register {
m.register {
'global-element',
- 'uppercase-local',
} {
group = 'strict-conventions',
severity = 'Warning',
diff --git a/test/diagnostics/init.lua b/test/diagnostics/init.lua
index a18e5cea..4025dd8c 100644
--- a/test/diagnostics/init.lua
+++ b/test/diagnostics/init.lua
@@ -86,5 +86,4 @@ end
require 'diagnostics.common'
require 'diagnostics.type-check'
-require 'diagnostics.global-element'
-require 'diagnostics.uppercase-local' \ No newline at end of file
+require 'diagnostics.global-element' \ No newline at end of file
diff --git a/test/diagnostics/uppercase-local.lua b/test/diagnostics/uppercase-local.lua
deleted file mode 100644
index fb8b726d..00000000
--- a/test/diagnostics/uppercase-local.lua
+++ /dev/null
@@ -1,66 +0,0 @@
-local config = require 'config'
-local util = require 'utility'
-
--- disable all default groups to make isolated tests
-config.set(nil, 'Lua.diagnostics.groupFileStatus',
-{
- ['ambiguity'] = 'None',
- ['await'] = 'None',
- ['codestyle'] = 'None',
- ['strict-conventions'] = 'None',
- ['duplicate'] = 'None',
- ['global'] = 'None',
- ['luadoc'] = 'None',
- ['redefined'] = 'None',
- ['strict'] = 'None',
- ['strong'] = 'None',
- ['type-check'] = 'None',
- ['unbalanced'] = 'None',
- ['unused'] = 'None'
-})
-
--- enable single diagnostic that is to be tested
-config.set(nil, 'Lua.diagnostics.neededFileStatus',
-{
- ['uppercase-local'] = 'Any!' -- override groupFileStatus
-})
-
--- check that global elements are not warned about
-TEST [[
-Var1 = "global"
-VAR2 = "global"
-
-local x = true
-local <!Y!> = false
-local <!Var!> = false
-]]
-
-TEST [[
-local function test1()
- print()
-end
-
-local function <!Test2!>()
- print()
-end
-]]
-
-TEST [[
-local function closure1()
- local elem1 = 1
- local <!Elem2!> = 2
-end
-
-local function <!Closure2!>()
- local elem1 = 1
- local <!Elem2!> = 2
-end
-]]
-
--- reset configurations
-config.set(nil, 'Lua.diagnostics.groupFileStatus',
-{})
-config.set(nil, 'Lua.diagnostics.neededFileStatus',
-{})
-config.set(nil, 'Lua.diagnostics.globals',
-{}) \ No newline at end of file