Fixed linter issues in `daemon/lua`, added more config tests (!400) · Merge requests · Knot projects / Knot Resolver

Merged Marek Vavrusa requested to merge daemon-lua-linter-fixes into master 7 years ago

Nov 24, 2017

Merge branch 'fix-obvious-linter-problems' into 'daemon-lua-linter-fixes' · 505adec4
Petr Špaček authored 7 years ago
```
fixed obvious linter problems

See merge request !401
```
505adec4
fixed obvious linter problems · 0489e21c
Marek Vavruša authored 7 years ago
```
there are two modules that couldn't work before:

* graphite
* ketcd
```
0489e21c

daemon/lua: fixed unused variables, whitespace, cleanup · 3162da83

Marek Vavruša authored 7 years ago

Also fixed improper promotion of `ffi` to global variable.

```
$ luacheck --codes daemon/lua/
Checking daemon/lua/config.lua                    OK
Checking daemon/lua/kres-gen.lua                  OK
Checking daemon/lua/kres.lua                      OK
Checking daemon/lua/sandbox.lua                   OK
Checking daemon/lua/trust_anchors.lua             OK
Checking daemon/lua/zonefile.lua                  OK

Total: 0 warnings / 0 errors in 6 files
```

3162da83

added basic config test for consts, depend on knot >= 2.4, cleanup · b90e6c71

Marek Vavruša authored 7 years ago

The library now depends on libknot >= 2.4.0 (in Debian stable),
this allows us to remove a dead code and sed-ing of the kres.lua

Added a basic config tests to check that constants still work,
and basic interface to rrsets still works after the change.

b90e6c71

Admin message

Fixed linter issues in `daemon/lua`, added more config tests

Merge request reports

Activity