-
Notifications
You must be signed in to change notification settings - Fork 42
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
dependency-submission triggered by tag on default branch does not update dependency results for repository #242
Comments
This message comes from GitHub. It's surprising that it doesn't resolve the tag to determine which commit/branch it belongs to. Can you share the content of the uploaded dependency graph file? |
Do you know how to get at it? I don't really want to make a workflow that cats or attaches the file just to do this... but I suppose I could if we need that. https://github.com/xenoterracide/gradle-semver/actions/runs/9405701870 |
If you pass a https://github.com/gradle/actions/blob/main/docs/dependency-submission.md#configuration-parameters |
@xenoterracide the submitted GitHub Dependency Graph file contains a Ideally, these values would be processed on submission: if a Git SHA belongs to the default branch it could then apply to the default branch, irrespective of the Git Ref that it was created from. This doesn't happen, so we'll need to do something in the action. Here's a workaround you could employ, assuming that this workflow will only be run for tags on the
A more sophisticated solution would involve checking if the ref was for a tag, then checking if the tag points to a commit on the default branch. But I'd first like confirmation that the simple solution works. |
I'm not the issue author or in any way affiliated with him, but I can confirm that this is working: |
finally got around to generating the json, which would have been really easy to do yourselves as this really requires a very simple workflow and I don't want to create a ton of tags for the sake o seeing what it does. Might be worth it to have a feature, or the default be submit and attach instead of submit or attach. |
this is what it looks like if I submit that env var
dependency-graph_full-full.json (2).zip https://github.com/xenoterracide/gradle-semver/actions/runs/9799510946 |
Apply the work-around from [1] to associate the dependency graph built from tags to the `main` branch. [1]: gradle/actions#242 (comment) Signed-off-by: Sebastian Schuberth <[email protected]>
Apply the work-around from [1] to associate the dependency graph built from tags to the `main` branch. [1]: gradle/actions#242 (comment) Signed-off-by: Sebastian Schuberth <[email protected]>
Current Behavior
A release is likely to be built of a tag which points to a commit on main
minimal details of workflow,
Expected Behavior
submission to a ref that points to a commit in the default branch should work.
Context (optional)
No response
Steps to Reproduce
obviously you'll have to add some actual gradle build steps, but it's not significant in this issue I think.
Gradle version
8.7
Build scan URL (optional)
No response
Your Environment (optional)
checkout peformed differently than normal due to actions/checkout#882
I'm not certain whether or not the dependency report should really be used on every build of the default branch vs an actual release. I guess for now it'll have to be every build of the default branch
The text was updated successfully, but these errors were encountered: