This message was deleted.
# community-supports
Slackbot
02/02/2023, 6:51 AMThis message was deleted.
p
Paul Merlin
02/02/2023, 7:03 AMNo it won't. The interpreter in 8.0 is a first cut and only supports what's described in the release notes. We have plans to improve it progressively though.
e
ephemient
02/02/2023, 7:04 AMthe link from the release notes to the documentation says that ``«plugin id»`` is part of the constrained syntax
p
Paul Merlin
02/02/2023, 7:05 AMNote that using version catalog aliases for plugins or plugin specification type-safe accessors is not supported by the plugins {} block interpreter. This support will be added in a later version.
Paul Merlin
02/02/2023, 7:05 AMMaybe adding examples of what is not supported would make this clearer?
➕ 2
e
ephemient
02/02/2023, 7:07 AMI guess it would be clearer to at least have a link from https://docs.gradle.org/8.0-rc-2/userguide/plugins.html#sec:constrained_syntax to somewhere describing other effects of using different syntax (such as this performance impact)
ephemient
02/02/2023, 7:07 AMbut it doesn't affect correctness, and is changing in the future… idk
p
Paul Merlin
02/02/2023, 7:08 AMExactly
Paul Merlin
02/02/2023, 7:14 AMWhat about https://github.com/gradle/gradle/pull/23739?
e
ephemient
02/02/2023, 7:15 AMLGTM. (also I didn't know
Copy code
...`...`...p
Paul Merlin
02/02/2023, 7:15 AMI had to look it up 🙂
Paul Merlin
02/02/2023, 7:16 AMPlease approve the PR if it LGTY
e
ephemient
02/02/2023, 7:18 AMI don't think I'm authorized to do anything but I'll at least comment 🙂
p
Paul Merlin
02/02/2023, 7:18 AManybody can do PR reviews
v
Vladimir Sitnikov
02/02/2023, 7:18 AMOne more question I had was
id("java-library") apply falsep
Paul Merlin
02/02/2023, 7:20 AMI answered on the PR
Paul Merlin
02/02/2023, 7:21 AMWe decided not to warn for now because it causes no "regression" nor correctness issue
Paul Merlin
02/02/2023, 7:22 AMWe might consider doing so once the interpreter supports all expected syntaxes, the warning would mean you're doing something funny
v
Vladimir Sitnikov
02/02/2023, 7:23 AMI agree warning by default is not something we want.
However, it might be fun to have something like
org.gradle.earlyadopter.plugins_block.kotlin_interpreter=truep
Paul Merlin
02/02/2023, 7:24 AMWe won't add this to 8.0, that ship has sailed. But thanks for the suggestion
👍 1
Paul Merlin
02/02/2023, 7:30 AM@Vladimir Sitnikov is your 👍 reaction on my comment to say that the current snippet is enough or that you would prefer it with all cases listed separately?
Paul Merlin
02/02/2023, 7:30 AMThanks for clarifying!
5 Views