-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Disallow setters for properties on parameter-like objects #9323
Comments
This issue has been automatically marked as stale because it has not had recent activity. Given the limited bandwidth of the team, it will be automatically closed if no further activity occurs. If you're interested in how we try to keep the backlog in a healthy state, please read our blog post on how we refine our backlog. If you feel this is something you could contribute, please have a look at our Contributor Guide. Thank you for your contribution. |
This issue has been automatically closed due to inactivity. If you can reproduce this on a recent version of Gradle or if you have a good use case for this feature, please feel free to reopen the issue with steps to reproduce, a quick explanation of your use case or a high-quality pull request. |
I think we still want to do this. |
Let's consider deprecating this for 7.0, too. |
This issue has been automatically marked as stale because it has not had recent activity. Given the limited bandwidth of the team, it will be automatically closed if no further activity occurs. If you're interested in how we try to keep the backlog in a healthy state, please read our blog post on how we refine our backlog. If you feel this is something you could contribute, please have a look at our Contributor Guide. Thank you for your contribution. |
This is a follow-up to #9068.
We want to encourage users to use mutable property objects instead of getter/setter pairs to configure parameter-like objects (tasks, artifact transforms, extensions, work items etc.).
So instead of this:
We'd want to support this:
Conversely, instead of naked types like this:
We'd like to migrate users towards
Property
types:We should thus issue warnings about any setter for a parameter value with a recommendation to use the corresponding mutable property type.
The text was updated successfully, but these errors were encountered: