[PR #62] [MERGED] Implement model, reader, wrapper for valve SKU sis files. #95

Closed
opened 2026-01-29 21:17:00 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/SabreTools/SabreTools.Serialization/pull/62
Author: @HeroponRikiBestest
Created: 1/27/2026
Status: Merged
Merged: 1/29/2026
Merged by: @mnadareski

Base: mainHead: steam-try-again


📝 Commits (8)

📊 Changes

9 files changed (+643 additions, -0 deletions)

View changed files

SabreTools.Serialization.Test/Readers/SkuSisTests.cs (+73 -0)
SabreTools.Serialization.Test/Wrappers/SkuSisTests.cs (+61 -0)
SabreTools.Serialization/Models/VDF/Constants.cs (+19 -0)
SabreTools.Serialization/Models/VDF/Sku.cs (+158 -0)
SabreTools.Serialization/Models/VDF/SkuSis.cs (+30 -0)
SabreTools.Serialization/Readers/SkuSis.cs (+146 -0)
📝 SabreTools.Serialization/WrapperFactory.cs (+11 -0)
SabreTools.Serialization/Wrappers/SkuSis.cs (+140 -0)
📝 SabreTools.Serialization/Wrappers/WrapperType.cs (+5 -0)

📄 Description

Re-PRing https://github.com/SabreTools/SabreTools.Serialization/pull/54 due to failed rebase.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/SabreTools/SabreTools.Serialization/pull/62 **Author:** [@HeroponRikiBestest](https://github.com/HeroponRikiBestest) **Created:** 1/27/2026 **Status:** ✅ Merged **Merged:** 1/29/2026 **Merged by:** [@mnadareski](https://github.com/mnadareski) **Base:** `main` ← **Head:** `steam-try-again` --- ### 📝 Commits (8) - [`0fa2991`](https://github.com/SabreTools/SabreTools.Serialization/commit/0fa2991d7d39d51bb705884365ded7fe39b71c02) Try again - [`199ab64`](https://github.com/SabreTools/SabreTools.Serialization/commit/199ab64b9cbdea8d0180ee4fed2b0d90dba206ee) Fix import alphebetization - [`9e4969a`](https://github.com/SabreTools/SabreTools.Serialization/commit/9e4969ab19260b8a5abe57070a5809c5daeea68c) Fixes. - [`3047688`](https://github.com/SabreTools/SabreTools.Serialization/commit/3047688d78c6c9831bcc6a270683f57de4c23f3a) first part of first attempt at a model - [`38484b0`](https://github.com/SabreTools/SabreTools.Serialization/commit/38484b0d6e7972bd40dfb41dd29cdad4d13d3618) Reimplement Sku Sis parsing - [`64626b1`](https://github.com/SabreTools/SabreTools.Serialization/commit/64626b1d38baf2fc99ec640f40ae05d740a3c022) First round of fixes - [`8995ab3`](https://github.com/SabreTools/SabreTools.Serialization/commit/8995ab3daad602ffd2ade880e4519f3adb44bb32) Make sure stream isn't closed - [`7fe4cc9`](https://github.com/SabreTools/SabreTools.Serialization/commit/7fe4cc90988dc29cc4e5c967c242d2f3314d2ca0) Missed this newline ### 📊 Changes **9 files changed** (+643 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `SabreTools.Serialization.Test/Readers/SkuSisTests.cs` (+73 -0) ➕ `SabreTools.Serialization.Test/Wrappers/SkuSisTests.cs` (+61 -0) ➕ `SabreTools.Serialization/Models/VDF/Constants.cs` (+19 -0) ➕ `SabreTools.Serialization/Models/VDF/Sku.cs` (+158 -0) ➕ `SabreTools.Serialization/Models/VDF/SkuSis.cs` (+30 -0) ➕ `SabreTools.Serialization/Readers/SkuSis.cs` (+146 -0) 📝 `SabreTools.Serialization/WrapperFactory.cs` (+11 -0) ➕ `SabreTools.Serialization/Wrappers/SkuSis.cs` (+140 -0) 📝 `SabreTools.Serialization/Wrappers/WrapperType.cs` (+5 -0) </details> ### 📄 Description Re-PRing https://github.com/SabreTools/SabreTools.Serialization/pull/54 due to failed rebase. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-29 21:17:00 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SabreTools/SabreTools.Serialization#95