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

feat(gha): support macos #428

Conversation

georgettica
Copy link

related to #413

not sure if someone did or didn't just though that as it's taking for so long I will just start and ask forgiveness if someone has already started and close mine

@georgettica
Copy link
Author

not suprisingly, tests fails on windows because I presume file schemas..

I will split this into 2 and do the windows in a different PR, as it requires code changes and I can give value by adding this macos validation now

@georgettica georgettica changed the title feat(gha): support windows and macos feat(gha): support windows and macos | splitting to 2 PR's Sep 29, 2023
will do it in a different PR :)
@georgettica
Copy link
Author

also, golangci got a timeout, might be a good idea to cache it to remove issues for the next build?

@georgettica georgettica changed the title feat(gha): support windows and macos | splitting to 2 PR's feat(gha): support windows Sep 29, 2023
@georgettica georgettica changed the title feat(gha): support windows feat(gha): support macos Sep 29, 2023
@georgettica
Copy link
Author

@github-actions
Copy link

github-actions bot commented Oct 1, 2023

☔ The latest upstream change (presumably these) made this pull request unmergeable. Please resolve the merge conflicts.

@github-actions github-actions bot added the needs rebase Pull request has conflicts and should be rebased label Oct 1, 2023
@gavv
Copy link
Owner

gavv commented Oct 1, 2023

Hi, thanks for PR, but we already had one: #426

@gavv gavv closed this Oct 1, 2023
@gavv gavv added the duplicate This issue or pull request already exists label Oct 1, 2023
@georgettica georgettica deleted the feat/georgettica/gha/support-windows-and-macos branch October 3, 2023 12:32
@georgettica
Copy link
Author

I am glad it's closed :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists needs rebase Pull request has conflicts and should be rebased
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants