warp-core

So the targeting options are still lined up with the modifier description box. 

> I sorry I couldn't respond earlier as I was a bit busy lately and thanks for the review. Please tell me what else needs to be done for getting...

It looks like `ShipManager::SatisfyingShips()` isn't considering a variant as the target model to require outfits. Locally, I've done: ```c++ if(requiresOutfits || takeOutfits || model->ModelName() != model->VariantName()) ``` Adding the third...

@tehhowch do you still want to review this?

> Will this take the ship regardless of how it is outfitted? > Will this take the ship even if the pilot no longer has one or more of the...

Anyone able to reproduce this?

> > Might it make sense to change "accept" to "ok" for dialogs that can't be declined? > > You are accepting a mission, so doesn't "accept" make more sense...

> It's weird that Codespell didn't catch these, right? Does that need to be checked? It'll only catch words that are in its dictionary (and aren't in our .exclude files.)...

> how does it look in-game? 

Any category that isn't listed in the `category "outfit"` list will actually work. At least, in the outfitter, and ship info panel. In the ship info display in the shipyard,...