-
Notifications
You must be signed in to change notification settings - Fork 0
Merge workflows #5
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
Conversation
502fa55
to
e2a3447
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left some comments
4e43508
to
32c6683
Compare
cli tests in windows using bash are failing due to max path issues, they dont fail using powershell as shell though:
The unique way i can think off is trying to reduce tests names to make them work in both shells as it is very inconvenient in general being not able to run them in a msys2 bash shell (Posted in cabal haskell#6271 (comment)) |
b2942b5
to
1f416e3
Compare
09175a7
to
eff0f11
Compare
160a1f4
to
db66ccb
Compare
Using validate job outputs cause it seems you can't use the env context directly in matrix
Please include the following checklist in your PR:
Please also shortly describe how you tested your change. Bonus points for added tests!