Skip to content

[Merged by Bors] - Fix ganache in windows CI#3105

Closed
realbigsean wants to merge 9 commits intosigp:unstablefrom
realbigsean:fix-ganache-ci
Closed

[Merged by Bors] - Fix ganache in windows CI#3105
realbigsean wants to merge 9 commits intosigp:unstablefrom
realbigsean:fix-ganache-ci

Conversation

@realbigsean
Copy link
Copy Markdown
Member

@realbigsean realbigsean commented Mar 21, 2022

Issue Addressed

Hopefully makes windows ganache installation more reliable.

Proposed Changes

  • use chocolatey to install windows build tools. This seems to often be the prescribed solution for node gyp issues. chocolatey is used here because npm install --global --production windows-build-tools hangs in github actions

Additional Info

I still haven't found why the prior installation technique would sometimes work, the windows-2019 environments seem to be identical across successes and failures. I think this should be re-run a few times to see if it can consistently pass

@realbigsean realbigsean changed the title check python version after attempting to set it in windows CI Fix ganache in windows CI Mar 21, 2022
@realbigsean realbigsean added the ready-for-review The code is ready for review label Mar 21, 2022
Copy link
Copy Markdown
Member

@michaelsproul michaelsproul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's try it

bors r+

bors bot pushed a commit that referenced this pull request Mar 21, 2022
## Issue Addressed

Hopefully makes windows ganache installation more reliable.

## Proposed Changes

- use `chocolatey` to install windows build tools. This seems to often be the prescribed solution for `node gyp` issues. `chocolatey` is used here because `npm install --global --production windows-build-tools` hangs in github actions

## Additional Info
I still haven't found why the prior installation technique would sometimes work, the `windows-2019` environments seem to be identical across successes and failures.  I think this should be re-run a few times to see if it can consistently pass


Co-authored-by: realbigsean <sean@sigmaprime.io>
@michaelsproul michaelsproul added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Mar 21, 2022
@bors bors bot changed the title Fix ganache in windows CI [Merged by Bors] - Fix ganache in windows CI Mar 22, 2022
@bors bors bot closed this Mar 22, 2022
divagant-martian pushed a commit to divagant-martian/lighthouse that referenced this pull request Mar 22, 2022
## Issue Addressed

Hopefully makes windows ganache installation more reliable.

## Proposed Changes

- use `chocolatey` to install windows build tools. This seems to often be the prescribed solution for `node gyp` issues. `chocolatey` is used here because `npm install --global --production windows-build-tools` hangs in github actions

## Additional Info
I still haven't found why the prior installation technique would sometimes work, the `windows-2019` environments seem to be identical across successes and failures.  I think this should be re-run a few times to see if it can consistently pass


Co-authored-by: realbigsean <sean@sigmaprime.io>
paulhauner pushed a commit to paulhauner/lighthouse that referenced this pull request May 6, 2022
## Issue Addressed

Hopefully makes windows ganache installation more reliable.

## Proposed Changes

- use `chocolatey` to install windows build tools. This seems to often be the prescribed solution for `node gyp` issues. `chocolatey` is used here because `npm install --global --production windows-build-tools` hangs in github actions

## Additional Info
I still haven't found why the prior installation technique would sometimes work, the `windows-2019` environments seem to be identical across successes and failures.  I think this should be re-run a few times to see if it can consistently pass


Co-authored-by: realbigsean <sean@sigmaprime.io>
@realbigsean realbigsean deleted the fix-ganache-ci branch November 21, 2023 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-merge This PR is ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants