Skip to content

Commit

Permalink
Bootsnap fixed blowing up on their side no need to mention it
Browse files Browse the repository at this point in the history
See #712 - thanks @fidalgo!

Also see Shopify/bootsnap#35
  • Loading branch information
PragTob committed Jun 25, 2019
1 parent 318b212 commit 9665035
Showing 1 changed file with 0 additions and 25 deletions.
25 changes: 0 additions & 25 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -137,17 +137,6 @@ to use SimpleCov with them. Here's an overview of the known ones:
<table>
<tr><th>Framework</th><th>Notes</th><th>Issue</th></tr>
<tr>
<th>
bootsnap
</th>
<td>
<a href="#want-to-use-bootsnap-with-simplecov">See section below.</a>
</td>
<td>
<a href="https://github.com/Shopify/bootsnap/issues/35">Shopify/bootsnap#35</a>
</td>
</tr>
<tr>
<th>
parallel_tests
Expand Down Expand Up @@ -646,20 +635,6 @@ Rails.application.eager_load!

Or you could remove `gem 'spring'` from your `Gemfile`.

## Want to use bootsnap with SimpleCov?

As mentioned in [this issue](https://github.com/Shopify/bootsnap/issues/35) iseq
loading/dumping doesn't work with coverage. Hence you need to deactivate it when
you run coverage so for instance when you use the environment `COVERAGE=true` to
decide that you want to gather coverage you can do:

```ruby
Bootsnap.setup(
compile_cache_iseq: !ENV["COVERAGE"], # Compile Ruby code into ISeq cache, breaks coverage reporting.
# all those other options
)
```

## Troubleshooting

The **most common problem is that simplecov isn't required and started before everything else**. In order to track coverage for your whole application **simplecov needs to be the first one** so that it (and the underlying coverage library) can subsequently track loaded files and their usage.
Expand Down

0 comments on commit 9665035

Please sign in to comment.