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

GitHub actions improvements #216

Closed
wants to merge 5 commits into from

Conversation

lukel97
Copy link
Contributor

@lukel97 lukel97 commented Jul 1, 2020

No description provided.

It was slow and is now covered by GitHub actions CI, which can run
these jobs in parallel
Copy link
Collaborator

@fendor fendor left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you!

@lukel97
Copy link
Contributor Author

lukel97 commented Jul 1, 2020

Ugh windows build is doing something flaky. Good to merge

@fendor
Copy link
Collaborator

fendor commented Jul 1, 2020

Why is it flaky? Is it a good idea to replace something working with a flaky build?

@@ -31,14 +30,30 @@ jobs:
path: ~/.cabal
Copy link
Member

Choose a reason for hiding this comment

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

does this path works for windows? The last build is not using cache but it doesn't save it afaics

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The path is working, I think it's this issue actions/cache#144

@fendor
Copy link
Collaborator

fendor commented Nov 18, 2020

Merged in #267

@fendor fendor closed this Nov 18, 2020
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