Skip to content

[PATCH v6] travis: move code style check to separate job#31

Closed
muvarov wants to merge 1 commit intoOpenDataPlane:masterfrom
muvarov:master_checkpatch_job
Closed

[PATCH v6] travis: move code style check to separate job#31
muvarov wants to merge 1 commit intoOpenDataPlane:masterfrom
muvarov:master_checkpatch_job

Conversation

@muvarov
Copy link
Contributor

@muvarov muvarov commented May 16, 2017

Move this check to separate job to better see which exactly
task was failed.

Signed-off-by: Maxim Uvarov maxim.uvarov@linaro.org

@muvarov muvarov changed the title travis: move code style check to separate job [PATCH v1] travis: move code style check to separate job May 16, 2017
@codecov-io
Copy link

codecov-io commented May 16, 2017

Codecov Report

Merging #31 into master will decrease coverage by 0.012%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##            master       #31       +/-   ##
=============================================
- Coverage   78.357%   78.345%   -0.013%     
=============================================
  Files          172       172               
  Lines        31485     31485               
=============================================
- Hits         24671     24667        -4     
- Misses        6814      6818        +4
Impacted Files Coverage Δ
platform/linux-generic/pktio/ipc.c 76.996% <0%> (-0.959%) ⬇️
...on_plat/validation/api/traffic_mngr/traffic_mngr.c 68.801% <0%> (-0.111%) ⬇️
test/common_plat/performance/odp_pktio_ordered.c 71.106% <0%> (+0.204%) ⬆️

@muvarov muvarov force-pushed the master_checkpatch_job branch from 4807b34 to c4ac7fc Compare May 16, 2017 15:37
@muvarov muvarov changed the title [PATCH v1] travis: move code style check to separate job [PATCH v2] travis: move code style check to separate job May 16, 2017
@muvarov muvarov force-pushed the master_checkpatch_job branch from c4ac7fc to de17c3c Compare May 16, 2017 15:41
@muvarov muvarov changed the title [PATCH v2] travis: move code style check to separate job [PATCH v3] travis: move code style check to separate job May 16, 2017
@muvarov muvarov force-pushed the master_checkpatch_job branch from de17c3c to f2fb2f0 Compare May 16, 2017 15:45
@muvarov muvarov changed the title [PATCH v3] travis: move code style check to separate job [PATCH v4] travis: move code style check to separate job May 16, 2017
@muvarov muvarov force-pushed the master_checkpatch_job branch from f2fb2f0 to 647d8c7 Compare May 16, 2017 16:53
@muvarov muvarov changed the title [PATCH v4] travis: move code style check to separate job [PATCH v5] travis: move code style check to separate job May 16, 2017
@muvarov
Copy link
Contributor Author

muvarov commented May 22, 2017

@Bill-Fischofer-Linaro looks like travis was down. Test results with this patch will be like:
https://travis-ci.org/Linaro/odp/builds/232876596

Move this check to separate job to better see which exactly
task was failed.

Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
@muvarov muvarov force-pushed the master_checkpatch_job branch from 647d8c7 to ba3e0ee Compare May 22, 2017 19:38
@muvarov muvarov changed the title [PATCH v5] travis: move code style check to separate job [PATCH v6] travis: move code style check to separate job May 22, 2017
Copy link
Contributor

@Bill-Fischofer-Linaro Bill-Fischofer-Linaro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed-by: Bill Fischofer bill.fischofer@linaro.org

@muvarov
Copy link
Contributor Author

muvarov commented May 23, 2017

@Bill-Fischofer-Linaro looks like travis is not very stable but test result for this commit is here: https://travis-ci.org/muvarov/odp/builds/234950108

@muvarov
Copy link
Contributor Author

muvarov commented May 23, 2017

Merged.

@muvarov muvarov closed this May 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants