Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: add mac to github workflows #9

Merged
merged 1 commit into from
Sep 21, 2023
Merged

Conversation

jericht
Copy link
Contributor

@jericht jericht commented Aug 31, 2023

What was the problem/requirement? (What/Why)

We should be building & testing our code on Windows and Mac

What was the solution? (How)

Modify GitHub workflows so they are building & testing our code on Windows and Mac. Made similar changes done in aws-deadline/deadline-cloud@afef801#diff-a1a47fe3f012a1f3ed362997e7d812693102beecce28d3eebbd7f446795d84d5

What is the impact of this change?

GitHub workflows are building & testing our code on Windows and Mac

How was this change tested?

GitHub workflows are building & testing our code on Windows and Mac

Was this change documented?

No

Is this a breaking change?

No

🍎 🪟 🍔

@jericht jericht changed the title chore: add windows and mac to github workflows chore: add mac to github workflows Sep 5, 2023
@jericht
Copy link
Contributor Author

jericht commented Sep 5, 2023

Removed Windows CI because adaptor runtime is using Unix sockets which fails the tests on Windows as that's not available

@jericht jericht marked this pull request as ready for review September 20, 2023 22:12
@jericht jericht requested a review from a team as a code owner September 20, 2023 22:12
Signed-off-by: Jericho Tolentino <68654047+jericht@users.noreply.github.com>
@jericht jericht merged commit ff0ce19 into mainline Sep 21, 2023
8 checks passed
@jericht jericht deleted the jericht/macwindows_burger branch September 21, 2023 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants