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

Speedup CI build somewhat #171

Merged
merged 2 commits into from
Jul 19, 2022
Merged

Speedup CI build somewhat #171

merged 2 commits into from
Jul 19, 2022

Conversation

oyvindberg
Copy link
Owner

the ENV: CI was lost at some point, and with it the sneaky coursier cache where the snapshot tests place resolving cache within the coursier folder.

there is still more to be done here

this is only needed because it makes some windows development stuff available. figure out how bleep can download it itself
@oyvindberg oyvindberg changed the title Speedup build somewhat Speedup CI build somewhat Jul 19, 2022
@oyvindberg oyvindberg merged commit c3067d8 into master Jul 19, 2022
@oyvindberg oyvindberg deleted the tweak-build branch July 19, 2022 23:11
@@ -29,6 +29,8 @@ jobs:
run: cs install --channel channel bleep --verbose

- name: Run tests
env:
CI: true
Copy link
Contributor

@sideeffffect sideeffffect Jul 25, 2022

Choose a reason for hiding this comment

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

Copy link
Owner Author

Choose a reason for hiding this comment

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

Yeah, that's probably why I deleted the declarations in the first place, heh :)

I was trying to get the first branch taken here, but it somehow still didn't work. I just merged this branch without digging too much, as it'll need to be revisited very soon anyway.

The integration snapshot tests imports a bunch of real, complex sbt builds into bleep, and makes a bunch of property tests on the result. Currently all these other builds are git submodules which need to be checked out for every build, and coursier needs to resolve all the dependencies for all the builds. as a result the github action cache is absolutely enormous. My plan is to replace submodules with code which checks out the projects when needed, and try to wrestle coursier into resolving dependencies without actually downloading them.

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.

2 participants