Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove update() from public API in 8.6 #27722

Closed
mlopatkin opened this issue Jan 17, 2024 · 0 comments · Fixed by #27726
Closed

Remove update() from public API in 8.6 #27722

mlopatkin opened this issue Jan 17, 2024 · 0 comments · Fixed by #27726
Assignees
@mlopatkin
Labels
a:chore Minor issue without significant impact in:provider-api property lazy provider MapProperty ListProperty DirectoryProperty
Milestone
8.6 RC3

Comments

@mlopatkin
Copy link
Member

The Property update API addresses an important use case but is very close to the proposed withActualValue (provisional name) mutation API in nature. After a discussion, we realized that we might need reconsider names and the overall shape of both. To allow more flexibility we should remove update from the public API of the coming release, so the users do not adopt it in a preliminary form.
@mlopatkin mlopatkin self-assigned this Jan 17, 2024
@mlopatkin mlopatkin added a:chore Minor issue without significant impact in:provider-api property lazy provider MapProperty ListProperty DirectoryProperty and removed to-triage labels Jan 17, 2024
@mlopatkin mlopatkin added this to the 8.6 RC3 milestone Jan 17, 2024
@mlopatkin mlopatkin mentioned this issue Jan 17, 2024
Merged
@mlopatkin mlopatkin linked a pull request Jan 17, 2024 that will close this issue
Remove update from public API of 8.6 #27726
Merged
This was referenced Feb 5, 2024
Merged
Closed
Merged
Merged
@github-actions github-actions bot mentioned this issue Mar 1, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mlopatkin mlopatkin

Labels
a:chore Minor issue without significant impact in:provider-api property lazy provider MapProperty ListProperty DirectoryProperty
Projects
None yet
Milestone
8.6 RC3
Development

Successfully merging a pull request may close this issue.

Remove update from public API of 8.6 gradle/gradle
1 participant
@mlopatkin