[test] Introduce Github action for CodeChecker tests #3066
Conversation
| name: codechecker-tests | ||
|
|
||
| # Triggers the workflow on push or pull request events. | ||
| on: [push, pull_request] |
There was a problem hiding this comment.
are there triggers for merging to master master branch?
There was a problem hiding this comment.
No, there is no such option. See: https://gh.neting.ccmunity/t/trigger-workflow-only-on-pull-request-merge/17359/3. It says that on merge event the push event also will be called.
| run: | | ||
| pip install $(grep -iE "pylint|pycodestyle" analyzer/requirements_py/dev/requirements.txt) | ||
| - name: Run tests | ||
| run: make pylint pycodestyle |
There was a problem hiding this comment.
are there any javascript linters?
csordasmarton
force-pushed
the
test_github_action
branch
from
d2f6c60 to
832ae87
Compare
csordasmarton
force-pushed
the
test_github_action
branch
8 times, most recently
from
ebfe362 to
358c39c
Compare
There was a problem hiding this comment.
My comment for #3066 (comment) comment: we run the JavaScript linters in the GUI related job. For this we have to install node and also run the npm install command which takes too much time. In the gui job it is necessary to install these, so this is the reason why I moved GUI linter test to this job and not to the separate linter job.
| name: codechecker-tests | ||
|
|
||
| # Triggers the workflow on push or pull request events. | ||
| on: [push, pull_request] |
There was a problem hiding this comment.
No, there is no such option. See: https://gh.neting.ccmunity/t/trigger-workflow-only-on-pull-request-merge/17359/3. It says that on merge event the push event also will be called.
csordasmarton
force-pushed
the
test_github_action
branch
9 times, most recently
from
a10e9f6 to
4879ce4
Compare
csordasmarton
force-pushed
the
test_github_action
branch
from
4879ce4 to
c3e0510
Compare
No description provided.