Add contributing task #605
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dnestoro
requested review from
melix,
mhalbritter,
olpaw,
fniephaus and
vjovanov
|
Hey, I'm a bit lost how the workflow works: It asks me a question where my tests are located. What tests? Existing tests for the library I want to add? I thought it creates some test stubs which I have then to implement. I ran the task |
mhalbritter
reviewed
Mar 21, 2025
tests/tck-build-logic/src/main/java/org/graalvm/internal/tck/ContributionTask.java
Outdated
Show resolved
Hide resolved
|
Okay, after cleaning up some empty directories it now continues, but then crashes with: I'm using |
|
Okay, i retried with a GraalVM 17, and now it looks better. One thing to note: I first added metadata for 2.12, and then for 2.11. Now the metadata index.json looks like this: [ {
"metadata-version" : "2.12",
"tested-versions" : [ "2.12" ],
"module" : "de.mkammerer:argon2-jvm"
}, {
"metadata-version" : "2.11",
"tested-versions" : [ "2.11" ],
"latest" : true,
"module" : "de.mkammerer:argon2-jvm"
} ]I don't think the |
|
I also noticed in the {
"allowed-packages" : [ "de.mkammerer.argon2", "de.mkammerer" ],
"directory" : "de.mkammerer/argon2-jvm",
"module" : "de.mkammerer:argon2-jvm"
}I get where the |
|
Now I've tried to add tests which not only need JUnit, but also some more dependencies (in this case In the first run, I've omitted Awaitility just to see what happens, and it breaks (which I guess is expected): Then i reran [ {
"metadata-version" : "2.12",
"tested-versions" : [ "2.12" ],
"module" : "de.mkammerer:argon2-jvm"
}, {
"metadata-version" : "2.12",
"tested-versions" : [ "2.12" ],
"latest" : true,
"module" : "de.mkammerer:argon2-jvm"
} ]The first |
mhalbritter
reviewed
Mar 21, 2025
tests/tck-build-logic/src/main/java/org/graalvm/internal/tck/ContributionTask.java
Outdated
Show resolved
Hide resolved
tests/tck-build-logic/src/main/java/org/graalvm/internal/tck/ContributionTask.java
Outdated
Show resolved
Hide resolved
tests/tck-build-logic/src/main/java/org/graalvm/internal/tck/ContributionTask.java
Outdated
Show resolved
Hide resolved
tests/tck-build-logic/src/main/java/org/graalvm/internal/tck/ContributionTask.java
Outdated
Show resolved
Hide resolved
tests/tck-build-logic/src/main/java/org/graalvm/internal/tck/ContributionTask.java
Outdated
Show resolved
Hide resolved
tests/tck-build-logic/src/main/java/org/graalvm/internal/tck/ContributionTask.java
Show resolved
Hide resolved
tests/tck-build-logic/src/main/java/org/graalvm/internal/tck/utils/FilesUtils.java
Outdated
Show resolved
Hide resolved
tests/tck-build-logic/src/main/resources/contributing/questions.json
Outdated
Show resolved
Hide resolved
tests/tck-build-logic/src/main/resources/contributing/questions.json
Outdated
Show resolved
Hide resolved
tests/tck-build-logic/src/main/resources/contributing/questions.json
Outdated
Show resolved
Hide resolved
this issue is already reported here |
It came from the |
@mhalbritter I am now throwing an exception in case you want to provide support for the library + same version of it, if it already exists. See this |
melix
reviewed
Mar 24, 2025
There was a problem hiding this comment.
I didn't take a look yet, just something that was obvious in the first place: you are invoking CLI in many places, but you shouldn't do this. You are running as a Gradle plugin, so you should use the Gradle constructs which are cross platform, e.g FileSystemOperations.
|
@melix @mhalbritter thanks for the feedback. I will refactor the PR to avoid using CLI, and ping you again when the PR is ready for review again. Until then, I will move this PR into a draft state, so please ignore incoming changes and don't test it until I finish refactoring |
dnestoro
force-pushed
the
dnestoro/add-contributing-task
branch
from
0f8780c to
26bed69
Compare
mhalbritter
approved these changes
May 5, 2025
dnestoro
force-pushed
the
dnestoro/add-contributing-task
branch
from
8180a34 to
1b59e9d
Compare
olpaw
reviewed
May 19, 2025
dnestoro
force-pushed
the
dnestoro/add-contributing-task
branch
from
6e98bf1 to
518bf33
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Adds interactive task that helps users to contribute to the metadata repository. The user only needs to provide path to tests and answer few questions in order to make a fully usable pull request.
For reviewers: