search

elastic / ecs-logging-php

Apache License 2.0
19 stars 35 forks source link

ci(jenkins): fixes sanity check #6

Closed v1v closed 4 years ago

v1v commented 4 years ago

I guess, we might need to fix the shared library to ensure the env variable is passed otherwise:

18:06:28  + python -
18:06:28  + curl https://pre-commit.com/install-local.py
18:06:28    % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
18:06:28                                   Dload  Upload   Total   Spent    Left  Speed
18:06:28  
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100  2590  100  2590    0     0  11511      0 --:--:-- --:--:-- --:--:-- 11511
18:06:28  Traceback (most recent call last):
18:06:28    File "<stdin>", line 103, in <module>
18:06:28    File "<stdin>", line 64, in main
18:06:28    File "/usr/lib/python2.7/UserDict.py", line 40, in __getitem__
18:06:28      raise KeyError(key)
18:06:28  KeyError: u'HOME'
v1v commented 4 years ago

This particular PR is not required as the precommit step is the one that should be in charge to configure the required variables