Skip to content

Test coverage for 32-bit too#4699

Closed
wilzbach wants to merge 1 commit intodlang:masterfrom
wilzbach:test_32
Closed

Test coverage for 32-bit too#4699
wilzbach wants to merge 1 commit intodlang:masterfrom
wilzbach:test_32

Conversation

@wilzbach
Copy link
Contributor

in reference to #4639

@9il
Copy link
Member

9il commented Jul 31, 2016

@wilzbach where is the GSoC plan? 👿

@JackStouffer
Copy link
Contributor

Why? Travis is not for testing Phobos. We have the auto tester for that.

@wilzbach
Copy link
Contributor Author

Why? Travis is not for testing Phobos.

Jobs are run parallel at Travis and the idea of adding the 32-bit job is that we go through all 32-bits paths at least once and don't have misleading coverage reports.

We have the auto tester for that.

The main reason for Travis is that it's a lot easier to modify - if you want to integrate coverage statistic with AutoTester (e.g. with codecov-bash) that would be very nice too :)

@codecov-io
Copy link

Current coverage is 88.71% (diff: 100%)

Merging #4699 into master will increase coverage by 0.01%

@@             master      #4699   diff @@
==========================================
  Files           121        122     +1   
  Lines         73916      74163   +247   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits          65566      65793   +227   
- Misses         8350       8370    +20   
  Partials          0          0          

Powered by Codecov. Last update d7a7173...41c657b

@wilzbach wilzbach changed the title [Do not merge] Test on Travis with 32-bit too Test coverage for 32-bit too Jul 31, 2016
@wilzbach
Copy link
Contributor Author

wilzbach commented Aug 8, 2016

Superseded by #4719

@wilzbach wilzbach closed this Aug 8, 2016
@wilzbach wilzbach deleted the test_32 branch August 8, 2016 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants