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

Remove the dependency collector and add a global additionalRuntimeClasspath configuration #84

Merged
merged 2 commits into from
Jul 6, 2024

Conversation

Matyrobbrt
Copy link
Member

No description provided.

@Matyrobbrt Matyrobbrt added bug Something isn't working enhancement New feature or request labels Jul 6, 2024
@neoforged-pr-publishing
Copy link

neoforged-pr-publishing bot commented Jul 6, 2024

  • Publish PR to GitHub Packages

Last commit published: 03aad35da1f64e058be032cf386cb9cb0cefbe27.

PR Publishing

The artifacts published by this PR:

Repository Declaration

In order to use the artifacts published by the PR, add the following repository to your buildscript:

repositories {
    maven {
        name 'Maven for PR #84' // https://github.com/neoforged/ModDevGradle/pull/84
        url 'https://prmaven.neoforged.net/ModDevGradle/pr84'
        content {
            includeModule('net.neoforged', 'moddev-gradle')
            includeModule('net.neoforged.moddev', 'net.neoforged.moddev.gradle.plugin')
            includeModule('net.neoforged.moddev.repositories', 'net.neoforged.moddev.repositories.gradle.plugin')
        }
    }
}

@Matyrobbrt Matyrobbrt changed the title Fix the run dependency collector and add a global additionalRuntimeClasspath configuration Remove the dependency collector and add a global additionalRuntimeClasspath configuration Jul 6, 2024
Copy link
Member

@Technici4n Technici4n left a comment

Choose a reason for hiding this comment

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

You can merge without fixing the configuration name if you don't want to. We should also add some documentation for this feature eventually.

@@ -320,6 +320,12 @@ public void apply(Project project) {
task.dependsOn(createArtifacts);
});

var additionalClasspath = configurations.create("additionalRuntimeClasspath", spec -> {
spec.setDescription("Contains dependencies of every run, that should not be considered boot classpath modules.");
Copy link
Member

Choose a reason for hiding this comment

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

I know that it's copy-pasted, but it's really not a great description. 😓

@Technici4n Technici4n merged commit da47655 into main Jul 6, 2024
4 checks passed
@Technici4n Technici4n deleted the fix-dep-collector branch July 6, 2024 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants