Skip to content

Move downloads from DownloadService to using Gradle Configurations#90

Closed
jpenilla wants to merge 1 commit into
PaperMC:mainfrom
jpenilla:configuration-downloads
Closed

Move downloads from DownloadService to using Gradle Configurations#90
jpenilla wants to merge 1 commit into
PaperMC:mainfrom
jpenilla:configuration-downloads

Conversation

@jpenilla
Copy link
Copy Markdown
Member

@jpenilla jpenilla commented Oct 8, 2021

Kind of hacky but seems to work pretty well

edit: needs changes to work with configuration cache

@jpenilla jpenilla requested a review from DenWav October 8, 2021 10:16
@jpenilla jpenilla linked an issue Oct 8, 2021 that may be closed by this pull request
@jpenilla jpenilla force-pushed the configuration-downloads branch 5 times, most recently from 13eaac0 to b9fa720 Compare October 13, 2021 07:17
@MiniDigger
Copy link
Copy Markdown
Member

not really in a position to review this, @DenWav, you said you have ideas for this too, do you like this approach?
maybe resident gradle pro @octylFractal can take a look too?

@jpenilla jpenilla force-pushed the configuration-downloads branch 3 times, most recently from 85e1f29 to 459b2d6 Compare October 19, 2021 20:00
@jpenilla jpenilla force-pushed the configuration-downloads branch from 459b2d6 to ae3f7d4 Compare October 20, 2021 19:58
@jpenilla jpenilla marked this pull request as draft October 22, 2021 00:34
@jpenilla jpenilla closed this Apr 29, 2024
@jpenilla jpenilla deleted the configuration-downloads branch April 29, 2024 02: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.

Investigate methods of moving dependency downloads to configurations

2 participants