Skip to content

Update PersistentConfiguration to use listeners for suppliers - #141

Open
kcooney wants to merge 3 commits into
mainfrom
kcooney/persisted-configuration-use-listener
Open

kcooney wants to merge 3 commits into
mainfrom
kcooney/persisted-configuration-use-listener

Conversation

@kcooney

@kcooney kcooney commented Feb 14, 2026

Copy link
Copy Markdown
Contributor

With this change, getting preference values via a supplier does not hit network
table APIs (instead, it is just a read on an atomic object).

@kcooney
kcooney force-pushed the kcooney/persisted-configuration-use-listener branch 2 times, most recently from d0a46e9 to dc8145f Compare February 14, 2026 17:42
@kcooney
kcooney marked this pull request as ready for review May 10, 2026 17:34
@kcooney
kcooney force-pushed the kcooney/persisted-configuration-use-listener branch 6 times, most recently from 78bedde to e12f832 Compare June 11, 2026 03:43
kcooney and others added 3 commits September 13, 2026 11:52
I noticed the WPILib Preferences test uses this annotation, so trying
it out to see if it reduces the crashes in the test runs.
@kcooney
kcooney force-pushed the kcooney/persisted-configuration-use-listener branch from e12f832 to 61ff02f Compare September 13, 2026 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant