Skip to content

Conversation

mikeland73
Copy link
Contributor

@mikeland73 mikeland73 commented Mar 1, 2023

Summary

This fixes mac os tests by setting NIX_PATH on the test itself. This should not be needed so it makes me a bit nervous we are covering up another issue.

Added auto-install test for mac

How was it tested?

CICD

@mikeland73 mikeland73 changed the title Landau/fix mac tests [CICD] Fix mac tests Mar 1, 2023
@mikeland73 mikeland73 force-pushed the landau/fix-mac-tests branch from d75b354 to dea9b5a Compare March 1, 2023 21:44
@mikeland73 mikeland73 requested review from gcurtis and loreto March 1, 2023 21:46
@mikeland73 mikeland73 marked this pull request as ready for review March 1, 2023 21:46
@mikeland73 mikeland73 force-pushed the landau/fix-mac-tests branch from dea9b5a to 59f82da Compare March 1, 2023 22:38
Copy link
Contributor

@loreto loreto left a comment

Choose a reason for hiding this comment

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

LGTM

@mikeland73 mikeland73 merged commit e12d3b6 into main Mar 1, 2023
@mikeland73 mikeland73 deleted the landau/fix-mac-tests branch March 1, 2023 23:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants