danger / peril

☢️ Serious and immediate danger.
https://danger.systems
MIT License
460 stars 58 forks source link

fix passing peril instance to inline runner #475

Closed pieh closed 3 years ago

pieh commented 3 years ago

This tries to fix the logic around appending "Peril context" before executing inline runner. Current logic checks for sandbox property but this is never set, so appending peril context is never executed.

From user perspective - this should fix accessing environment variables via peril.env.*, which is currently not working (peril objects is just {}, it doesn't even have .env field)

Closes https://github.com/danger/peril/issues/474

orta commented 3 years ago

Cool, yeah, looks good to me 👍🏻