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

Comment out windows jobs #364

Merged
merged 1 commit into from
May 5, 2022
Merged

Conversation

akrmn
Copy link
Contributor

@akrmn akrmn commented May 5, 2022

CI fails on windows due to commercialhaskell/stack#5732

@shayne-fletcher, could you please rebase your branch on master once this is merged? Alternatively, you could also cherry-pick the single commit here. I'll rerun CI on your PR once this is merged

@garyverhaegen-da
Copy link
Contributor

No need for a rebase, running CI again should be enough.

@garyverhaegen-da garyverhaegen-da enabled auto-merge (squash) May 5, 2022 14:36
Copy link

@sofiafaro-da sofiafaro-da left a comment

Choose a reason for hiding this comment

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

Thank you @akrmn and @garyverhaegen-da !

@garyverhaegen-da
Copy link
Contributor

For future audits, CI results on #363 evidence that this is a "rest of the internet broke" kind of failure, not one introduced in any of @shayne-fletcher's changes (in case there was any doubt).

One could argue we should look into pinning our dependencies, but given this is the fist time this has been an issue (that I am aware of), I don't think that's a high priority.

@garyverhaegen-da garyverhaegen-da merged commit 05e228c into master May 5, 2022
@garyverhaegen-da garyverhaegen-da deleted the comment-out-windows-ci branch May 5, 2022 15:52
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.

None yet

3 participants