-
Notifications
You must be signed in to change notification settings - Fork 834
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Lint licenses #1995
Lint licenses #1995
Conversation
/cc @axsaucedo @cliveseldon @RafalSkolasinski |
Mon Jun 22 12:46:26 UTC 2020 impatient try |
Mon Jun 22 12:47:57 UTC 2020 impatient try |
Mon Jun 22 15:27:05 UTC 2020 impatient try |
Mon Jun 22 15:28:37 UTC 2020 impatient try |
/test this |
Tue Jun 23 08:16:10 UTC 2020 impatient try |
16f6282
to
c3590d4
Compare
Tue Jun 23 08:45:50 UTC 2020 impatient try |
Tue Jun 23 08:45:57 UTC 2020 impatient try |
Tue Jun 23 09:11:04 UTC 2020 impatient try |
Tue Jun 23 09:11:08 UTC 2020 impatient try |
Tue Jun 23 15:57:52 UTC 2020 impatient try |
Tue Jun 23 15:57:57 UTC 2020 impatient try |
Tue Jun 23 16:12:46 UTC 2020 impatient try |
Tue Jun 23 16:13:45 UTC 2020 impatient try |
Thu Jun 25 09:33:03 UTC 2020 impatient try |
Thu Jun 25 09:33:16 UTC 2020 impatient try |
\lgtm |
/lgtm |
Mon Jun 29 08:55:44 UTC 2020 impatient try |
Tue Jul 7 15:47:41 UTC 2020 impatient try |
Tue Jul 7 15:47:45 UTC 2020 impatient try |
Tue Jul 7 15:50:35 UTC 2020 impatient try |
Tue Jul 7 15:50:36 UTC 2020 impatient try |
/unhold |
/hold cancel |
Wed Jul 8 08:41:34 UTC 2020 impatient try |
Wed Jul 8 08:41:39 UTC 2020 impatient try |
failed to trigger Pull Request pipeline
|
Fixes #1932
Changelog
make run_lint
andmake run_linter_check
tomake fmt
andmake lint
.make lint
to operator and executor (and add them to JX lint pipeline).make lint
checks if licenses have changed.Notes
On Python, lint seems to fail because the output ofpip-licenses
depends on the environment. Therefore, JX is generating a slightly different license doc. I've opened up an issue onpip-licenses
reporting this: raimon49/pip-licenses#69In the meantime, we can generate the output locally so that it matches what gets generated in the JX cluster.The issue with
pip-licenses
has now been resolved.