-
Notifications
You must be signed in to change notification settings - Fork 102
Build scans are not captured when GE plugin is applied within settingsEvaluated
#626
Comments
Interesting. Does this happen consistently for the project? If so, can you try to reproduce by adding these init scripts to the |
Yes, it is happening on all projects on different workflows. I am using reusable workflows, not sure if that can be the problem. I don't know what do you mean about reproducing it locally, I haven't developed any GitHub Action before 🤔 |
The way the action captures the build scan is via some scripts placed in the Gradle User Home So to reproduce:
|
There is a json file: GRADLE_ACTION-1680031868132.json {
"rootProjectName": "gradle-extensions-project",
"rootProjectDir": "D:\\Repos\\gradle\\gradle-extensions",
"requestedTasks": "assemble",
"gradleVersion": "8.0.2",
"gradleHomeDir": "C:\\Users\\javøie\\.gradle\\wrapper\\dists\\gradle-8.0.2-bin\\25jlreiuz6u3xu2phlpa2vv4m\\gradle-8.0.2",
"buildFailed": false,
"buildScanUri": "https://gradle.com/s/bai43tyy44txq",
"buildScanFailed": false
} |
Thanks. I think the fact that this file generates for you locally (with a |
The `PluginManager.hasPlugin` method was not detecting the GE plugin when it was applied by a separate custom plugin. Switching to `PluginManager.withPlugin` fixes this. Fixes #626
The `PluginManager.hasPlugin` method was not detecting the GE plugin when it was applied during settingsEvaluated. Switching to `PluginManager.withPlugin` fixes this. Fixes #626
@JavierSegoviaCordoba I finally got around to investigating this. It turns out that the issue was due to the way we detect the While I think it's a bit odd to apply a settings plugin during Can you please try running your project with |
How should I apply it? |
Apologies, I missed the hyphen from the version: you can literally reference This will cause your workflow to run with the action code in the dd/issue-626 branch. |
Sorry I was referring to gradle enterprise and how to apply it on settings plugins |
There's a lot going on in your plugin(s), so it's hard to be definitive. |
It works, feels free to close it now or when the change is published :) |
Will do. Can you confirm that by "it works" you mean that you tried the Did you also try moving the plugin application outside of the |
About the scans, they appear in the summary. I moved it but I haven't tested it yet, I will share it here when the test is done! |
Applying the plugin as you have said works too, thank you 🙏 |
settingsEvaluated
In this run it is possible to see the scan in the logs
But in the summary it is not appearing:
https://github.com/JavierSegoviaCordoba/semver-gradle-plugin/actions/runs/4394281726#summary-11932819131
The text was updated successfully, but these errors were encountered: