Skip to content

Conversation

@realbigsean
Copy link
Member

Issue Addressed

N/A

Proposed Changes

I didn't realize the PORTABLE env variable is only picked up by install in the Makefile so we are still getting SIGILLs:

https://github.com/sigp/lighthouse/runs/1565004525?check_suite_focus=true

Additional Info

@realbigsean realbigsean added the ready-for-review The code is ready for review label Dec 16, 2020
@michaelsproul
Copy link
Member

Dang, nice try anyway

bors r+

bors bot pushed a commit that referenced this pull request Dec 16, 2020
## Issue Addressed

N/A

## Proposed Changes

I didn't realize the `PORTABLE` env variable is only picked up by `install` in the `Makefile` so we are still getting `SIGILL`s:

https://github.com/sigp/lighthouse/runs/1565004525?check_suite_focus=true

## Additional Info



Co-authored-by: realbigsean <seananderson33@gmail.com>
@bors
Copy link

bors bot commented Dec 17, 2020

@bors bors bot changed the title Revert "add caching to test suite (#2089)" [Merged by Bors] - Revert "add caching to test suite (#2089)" Dec 17, 2020
@bors bors bot closed this Dec 17, 2020
@michaelsproul
Copy link
Member

I like the symmetry in the issue numbers, 2098 reverts 2089 😁

@realbigsean realbigsean deleted the revert-test-suite-caching branch November 21, 2023 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-review The code is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants