@abaplint/cli 2.102.0 → 2.102.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/build/cli.js +9 -3
- package/package.json +2 -2
package/build/cli.js
CHANGED
|
@@ -48465,7 +48465,7 @@ class Registry {
|
|
|
48465
48465
|
}
|
|
48466
48466
|
static abaplintVersion() {
|
|
48467
48467
|
// magic, see build script "version.sh"
|
|
48468
|
-
return "2.102.
|
|
48468
|
+
return "2.102.1";
|
|
48469
48469
|
}
|
|
48470
48470
|
getDDICReferences() {
|
|
48471
48471
|
return this.ddicReferences;
|
|
@@ -67507,6 +67507,12 @@ class UnnecessaryReturn extends _abap_rule_1.ABAPRule {
|
|
|
67507
67507
|
badExample: `FORM hello1.
|
|
67508
67508
|
WRITE 'world'.
|
|
67509
67509
|
RETURN.
|
|
67510
|
+
ENDFORM.
|
|
67511
|
+
|
|
67512
|
+
FORM foo.
|
|
67513
|
+
IF 1 = 2.
|
|
67514
|
+
RETURN.
|
|
67515
|
+
ENDIF.
|
|
67510
67516
|
ENDFORM.`,
|
|
67511
67517
|
goodExample: `FORM hello2.
|
|
67512
67518
|
WRITE 'world'.
|
|
@@ -67539,11 +67545,11 @@ ENDFORM.`,
|
|
|
67539
67545
|
const fix = edit_helper_1.EditHelper.deleteStatement(file, prev);
|
|
67540
67546
|
issues.push(issue_1.Issue.atStatement(file, prev, message, this.getMetadata().key, this.getConfig().severity, fix));
|
|
67541
67547
|
}
|
|
67548
|
+
// note: ENDTRY is not checked, it will usually just make it an empty catch handler, which is also bad
|
|
67542
67549
|
const prevprev = statements[i - 2];
|
|
67543
67550
|
if (prev && prevprev
|
|
67544
67551
|
&& prevprev.get() instanceof Statements.Return
|
|
67545
|
-
&&
|
|
67546
|
-
|| prev.get() instanceof Statements.EndTry)) {
|
|
67552
|
+
&& prev.get() instanceof Statements.EndIf) {
|
|
67547
67553
|
const fix = edit_helper_1.EditHelper.deleteStatement(file, prevprev);
|
|
67548
67554
|
issues.push(issue_1.Issue.atStatement(file, prevprev, message, this.getMetadata().key, this.getConfig().severity, fix));
|
|
67549
67555
|
}
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@abaplint/cli",
|
|
3
|
-
"version": "2.102.
|
|
3
|
+
"version": "2.102.1",
|
|
4
4
|
"description": "abaplint - Command Line Interface",
|
|
5
5
|
"funding": "https://github.com/sponsors/larshp",
|
|
6
6
|
"bin": {
|
|
@@ -38,7 +38,7 @@
|
|
|
38
38
|
},
|
|
39
39
|
"homepage": "https://abaplint.org",
|
|
40
40
|
"devDependencies": {
|
|
41
|
-
"@abaplint/core": "^2.102.
|
|
41
|
+
"@abaplint/core": "^2.102.1",
|
|
42
42
|
"@types/chai": "^4.3.5",
|
|
43
43
|
"@types/glob": "^7.2.0",
|
|
44
44
|
"@types/minimist": "^1.2.2",
|