Skip to content
This repository was archived by the owner on Apr 1, 2025. It is now read-only.

Use source-repository-package to pin Git deps in cabal. #91

Merged
merged 4 commits into from
Jun 5, 2019

Conversation

patrickt
Copy link
Contributor

@patrickt patrickt commented Jun 4, 2019

Right now our CI build cycle time is ~20 minutes, mostly because we're
rebuilding our vendored packages every time. This should help, at
least to some degree, with that. We still have to vendor haskell-tree-sitter
manually until haskell/cabal#5536 is fixed.

Patrick Thomson added 2 commits June 4, 2019 18:59
Right now our CI build cycle time is ~20 minutes, mostly because we're
rebuilding our vendored packages every time. This should help, at
least to some degree, with that.
Copy link
Contributor

@robrix robrix left a comment

Choose a reason for hiding this comment

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

Just the one change requested, otherwise 👍

@@ -29,7 +29,7 @@ before_install:

install:
- cabal new-update
- cabal new-configure --enable-tests --write-ghc-environment-files=always
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we’re going to want the .ghc.environment files generated for the doctests to work reliably, tho this may be more of a problem for matrix builds (IIRC).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Aha!

source-repository-package
type: git
location: https://github.com/rewinfrey/hspec-expectations-pretty-diff
tag: 94af5871c24ba319f7f72fefa53c1a4d074c9a29
Copy link
Contributor

Choose a reason for hiding this comment

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

This is v. cool.

@patrickt patrickt merged commit f7bc9b8 into master Jun 5, 2019
@patrickt patrickt deleted the use-smarter-cabal-project-when-possible branch June 5, 2019 14:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants