Skip to content

Add backing fields to KSP API#2969

Open
jaschdoc wants to merge 10 commits into
mainfrom
backing-fields2873
Open

Add backing fields to KSP API#2969
jaschdoc wants to merge 10 commits into
mainfrom
backing-fields2873

Conversation

@jaschdoc

@jaschdoc jaschdoc commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator
  • Add test for type of backing field, e.g., should be MutableList<String>

Closes #2873

@jaschdoc jaschdoc force-pushed the backing-fields2873 branch 14 times, most recently from 6d41cde to 983e328 Compare June 11, 2026 12:26
@jaschdoc jaschdoc force-pushed the backing-fields2873 branch 9 times, most recently from 3810e2a to 21f29e8 Compare June 19, 2026 08:53
@jaschdoc jaschdoc force-pushed the backing-fields2873 branch 2 times, most recently from 44239e4 to 77e93b8 Compare June 23, 2026 11:30
@jaschdoc jaschdoc force-pushed the backing-fields2873 branch from 4b85fb9 to 8d915ce Compare June 24, 2026 13:39
jaschdoc added 2 commits June 24, 2026 15:50
These tests use Room which does not yet support backing fields.
@jaschdoc jaschdoc force-pushed the backing-fields2873 branch from 8d915ce to 2305a36 Compare June 24, 2026 13:50
@jaschdoc jaschdoc marked this pull request as ready for review June 24, 2026 15:42
@jaschdoc

Copy link
Copy Markdown
Collaborator Author

I have ignored the integration tests that include Room. I'm working on submitting a fix for Room, but the problem is circular, since Room depends on KSP, and our tests depend on Room. Thus, the tests should be disabled until Room has the proper fix and then we can update our tests to use the fixed version of Room and re-enable the tests.

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.

Add support for explicit backing fields

1 participant