bump version to 8.3.0-1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

fixup! import 8.3.0 source

commit build of 8.3.0

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

import 8.3.0 source

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

build upstream to umd

as we want to avoid requiring npm et al for every build, this is also
effectively vendoring all in, so easier to avoid supply-chain attacks

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

use worker_threads for linting

instead linting all files in the main thread, use worker threads
for that (4 by default) and add the '-t' switch to able to control that

a basic benchmark of eslint of pve-manager showed some performance
gains:

Benchmark #1: Current
  Time (mean ± σ):      6.468 s ±  0.116 s    [User: 9.803 s, System: 0.333 s]
  Range (min … max):    6.264 s …  6.647 s    10 runs

Benchmark #2: 2Threads
  Time (mean ± σ):      4.509 s ±  0.106 s    [User: 12.706 s, System: 0.530 s]
  Range (min … max):    4.335 s …  4.674 s    10 runs

Benchmark #3: 4Threads
  Time (mean ± σ):      3.471 s ±  0.033 s    [User: 16.390 s, System: 0.630 s]
  Range (min … max):    3.431 s …  3.542 s    10 runs

Benchmark #4: 8Threads
  Time (mean ± σ):      2.880 s ±  0.044 s    [User: 22.454 s, System: 0.938 s]
  Range (min … max):    2.813 s …  2.964 s    10 runs

Summary
  '8Threads' ran
    1.21 ± 0.02 times faster than '4Threads'
    1.57 ± 0.04 times faster than '2Threads'
    2.25 ± 0.05 times faster than 'Current'

after 8 threads, there were no real performance benefits since the
overhead to load the module seems to be the biggest factor.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

remove unnecessary eslint.js

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

ship proper nodejs module 'pve-eslint'

instead of concatenating the eslint module into our app.js, ship
a 'pve-eslint' module that exports the built eslint module

to do this, we have to leave the module type on 'umd' instead of
changing to 'var' so that nodejs can properly import it.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 7.28.0-1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

set cwd of CLIEngine to process.cwd()

it seems it now defaults to '/' as the current working dir, but we
assume the cwd of the process

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

switch to es2020 for bullseyse

at least in theory this should be enough

https://eslint.org/docs/user-guide/configuring/language-options#specifying-environments

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

actually build upstream release 7.28.0

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

import eslint 7.28.0

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

buildsys: change upload dist to bullseye

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

add small script to help extracting more relevant changes

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 7.18.0-1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch to ECMA version 2018

Mainly to enable the rest/spread operator (...variable).

Other new features include asynchronous iteration,
Promise.prototype.finally and additions to RegExp (look behind, named
capture groups, s flag)

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

import eslint 7.18.0

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 7.12.1-1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

import 7.12.1 upstream release

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

exit with error also on warnings

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

app: eslint fixes

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 7.2.0-1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

import and build new upstream release 7.2.0

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

ship an example config and extend-rules sample files

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

add HelpInfo variables to globals

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

no-mixed-operators: do not check basic math precedence

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

improve no-extra-parens rule

We use both, 'no-extra-parens' and 'no-mixed-operators' rules and
those can conflict. Thus follow eslints recommendation[0] and allow
parenthesis in nestedBinaryExpressions to avoid the linter
complaining either way. Do this also fo enforceForArrowConditionals
as there I saw the issue already in practice for me.

[0]: https://eslint.org/docs/rules/no-mixed-operators#rule-details

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

try to improve object-curly-newline

for me consistency on open/close is most important..

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

disable 'no-else-return' rule

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

disable "no-underscore-dangle" as used in explicit unused variables

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

disable semi-style for now

In generally we follow it but we want also to allow statements like:

let foo = bar
    ? 'baz'
    : 'foz'
    ;

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

adapt some rules

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

add --output-config CLI option

can help to get the default config for adaption

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

add possibiliity to extend base config over CLI

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

factor out path expansion

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

no-unused-vars: allow starting with _underscore

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

checkin build of 7.1.0

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

update to 7.1.0 sources

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 7.0.0-1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

upgrade to v7.0.0

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 7.0.0~alpha3-2

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

ignore unused 'me'

sometimes we want to have it just in case

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

print relative file path

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

re do output formatting, save some lines

Moving the message and the info if something is auto-fixable to a
single line, saving ~ 2/3rd of output lines.

The auto-fixable got replaced with a (*) marker, it's not ideal but
IMO OK; explain what it means in the summaries "X issues marked with
(*) could be auto-fixed"

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

eslint: autofixes

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

allow '!!' for boolean conversion

we use this many times, and it really should not be confusing,
also it is much shorter than: 'Boolean(foo)',

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 7.0.0~alpha3-1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

drop "lints cwd on no params" hint

This is a bit strange behaviour, especially if there's nothing to
lint and one gets just a spewed out nodejs backtracke.. It's not hard
to just to `eslint *.js` or the like..

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

rules: dot-location: set location parameter to property

who the heck makes the following syntax the default:
> foo.
>    bar().
>    baz();

So let's enforce the right thing™ and enforce the dot sticking to the
property, not the object, i.e.:

> foo
>    .bar().
>    .baz();

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

exit sanely if no args are given

I mean some seem to like python style throwing up in ones face on
every occasion.. I don't though..

nodes drops the original argv0 in favor of process.execPath and the
path to the JavaScript file being executed, so we need more than two
args to be OK.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

globals: sort and add PBS

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 7.0.0~alpha3

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

add dsc target, fix various packaging related stuff

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

first commit

includes a (minimal) working wrapper

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>