-
Notifications
You must be signed in to change notification settings - Fork 873
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
Fix Restlet v2 Message#getAttributes
calls
#6796
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any idea why testLatestDeps didn't catch this?
Honestly, no. It's even more surprising that the muzzle check did not catch it either -- maybe maven coordinates have changed? I'll check that |
// the project repositories need to be retrieved after evaluation, before that the list is just empty | ||
val projectRepositories = getProjectRepositories(project) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Turns out that the project repositories weren't taken into account (they need to be retrieved afterEvaluate
, before that the list is just empty); and since restlet is not available in maven central, muzzle check did nothing.
@@ -6,7 +6,9 @@ muzzle { | |||
pass { | |||
group.set("org.restlet") | |||
module.set("org.restlet") | |||
versions.set("[1.0.0, 1.2-M1)") | |||
// TODO: rename this module to restlet-1.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will do that in the next PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
…6937) Fixes #6932 Replaces #6933 @aaron-ai I finally understood what you meant in #6796 after seeing your PR - initially I thought you were talking about some custom extension, that's outside of this repository. I added back the maven central fallback to the muzzle check plugin: after investigating a bit, I found out that the repositories defined in `settings.dependencyResolutionManagement` are not propagated at all (no matter whether it's `afterEvaluate` or not) to the `project.repositories`, and that the gradle issue gradle/gradle#17295 is still valid.
…pen-telemetry#6937) Fixes open-telemetry#6932 Replaces open-telemetry#6933 @aaron-ai I finally understood what you meant in open-telemetry#6796 after seeing your PR - initially I thought you were talking about some custom extension, that's outside of this repository. I added back the maven central fallback to the muzzle check plugin: after investigating a bit, I found out that the repositories defined in `settings.dependencyResolutionManagement` are not propagated at all (no matter whether it's `afterEvaluate` or not) to the `project.repositories`, and that the gradle issue gradle/gradle#17295 is still valid.
…pen-telemetry#6937) Fixes open-telemetry#6932 Replaces open-telemetry#6933 @aaron-ai I finally understood what you meant in open-telemetry#6796 after seeing your PR - initially I thought you were talking about some custom extension, that's outside of this repository. I added back the maven central fallback to the muzzle check plugin: after investigating a bit, I found out that the repositories defined in `settings.dependencyResolutionManagement` are not propagated at all (no matter whether it's `afterEvaluate` or not) to the `project.repositories`, and that the gradle issue gradle/gradle#17295 is still valid.
…pen-telemetry#6937) Fixes open-telemetry#6932 Replaces open-telemetry#6933 @aaron-ai I finally understood what you meant in open-telemetry#6796 after seeing your PR - initially I thought you were talking about some custom extension, that's outside of this repository. I added back the maven central fallback to the muzzle check plugin: after investigating a bit, I found out that the repositories defined in `settings.dependencyResolutionManagement` are not propagated at all (no matter whether it's `afterEvaluate` or not) to the `project.repositories`, and that the gradle issue gradle/gradle#17295 is still valid.
Fixes #6563