RPCS3/discord-bot
1
0
Fork 0
mirror of https://github.com/RPCS3/discord-bot.git synced 2026-07-19 13:54:44 -04:00
Code Issues 20 Packages Projects Releases Wiki Activity

[PR #525] [MERGED] Rewrite log parser to properly support multi-value items #595

New Issue
Closed
opened 2026-02-15 15:56:23 -05:00 by yindo · 0 comments
yindo commented 2026-02-15 15:56:23 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/RPCS3/discord-bot/pull/525
Author: @13xforever
Created: 3/2/2020
Status: ✅ Merged
Merged: 3/2/2020
Merged by: @13xforever

Base: master ← Head: vnext

📝 Commits (9)

  • 9fe21dc update deps
  • 424242e unify fatal error extraction
  • 9a47809 log parser was skipping the very last line of the log
  • f065a50 change note priority for controller binding
  • 23f706c fix license extractor for the new logs
  • 639bdd2 wip
  • 0d5a838 introduce proper multivalue collections for log parser (WIP)
  • 8e94a94 new patch extraction
  • ff40216 Merge branch 'multi_value_parser' into vnext

📊 Changes

14 files changed (+291 additions, -106 deletions)

View changed files

📝 Clients/AppveyorClient/AppveyorClient.csproj (+1 -1)
📝 Clients/GithubClient/GithubClient.csproj (+1 -1)
📝 Clients/PsnClient/PsnClient.csproj (+1 -1)
📝 CompatBot/CompatBot.csproj (+12 -12)
📝 CompatBot/EventHandlers/LogParsing/LogParser.LogSections.cs (+15 -13)
📝 CompatBot/EventHandlers/LogParsing/LogParser.PipeReader.cs (+4 -1)
📝 CompatBot/EventHandlers/LogParsing/LogParser.StateMachineGenerator.cs (+1 -6)
📝 CompatBot/EventHandlers/LogParsing/POCOs/LogParseState.cs (+3 -0)
📝 CompatBot/EventHandlers/UnknownCommandHandler.cs (+2 -1)
➕ CompatBot/Utils/NameUniqueObjectCollection.cs (+80 -0)
📝 CompatBot/Utils/ResultFormatters/LogParserResultFormatter.GeneralNotesSection.cs (+30 -29)
📝 CompatBot/Utils/ResultFormatters/LogParserResultFormatter.WeirdSettingsSection.cs (+16 -15)
📝 CompatBot/Utils/ResultFormatters/LogParserResultFormatter.cs (+26 -26)
➕ CompatBot/Utils/UniqueList.cs (+99 -0)

📄 Description

This will reduce memory footprint and speed up processing in checks

🔄 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/RPCS3/discord-bot/pull/525 **Author:** [@13xforever](https://github.com/13xforever) **Created:** 3/2/2020 **Status:** ✅ Merged **Merged:** 3/2/2020 **Merged by:** [@13xforever](https://github.com/13xforever) **Base:** `master` ← **Head:** `vnext` --- ### 📝 Commits (9) - [`9fe21dc`](https://github.com/RPCS3/discord-bot/commit/9fe21dcc20f4c8d4d98a7f885d858cb10b86332a) update deps - [`424242e`](https://github.com/RPCS3/discord-bot/commit/424242ed3157226f2ccb0b4220ad157162618ce0) unify fatal error extraction - [`9a47809`](https://github.com/RPCS3/discord-bot/commit/9a47809f847092abad46262c45cfc9100f8694e3) log parser was skipping the very last line of the log - [`f065a50`](https://github.com/RPCS3/discord-bot/commit/f065a500f852f873c2c0f8694b787c736a012832) change note priority for controller binding - [`23f706c`](https://github.com/RPCS3/discord-bot/commit/23f706c98f18bf9cba57689e12bd3dd25526cf33) fix license extractor for the new logs - [`639bdd2`](https://github.com/RPCS3/discord-bot/commit/639bdd280c45478097fa24fc0ef88782103f2eac) wip - [`0d5a838`](https://github.com/RPCS3/discord-bot/commit/0d5a838d2af8d6d6a87329a3119a9d063ec35b37) introduce proper multivalue collections for log parser (WIP) - [`8e94a94`](https://github.com/RPCS3/discord-bot/commit/8e94a94a82b2e66520517b36e14d3fb5028cff18) new patch extraction - [`ff40216`](https://github.com/RPCS3/discord-bot/commit/ff402160d211a968479f900bee0610ee92a6181b) Merge branch 'multi_value_parser' into vnext ### 📊 Changes **14 files changed** (+291 additions, -106 deletions) <details> <summary>View changed files</summary> 📝 `Clients/AppveyorClient/AppveyorClient.csproj` (+1 -1) 📝 `Clients/GithubClient/GithubClient.csproj` (+1 -1) 📝 `Clients/PsnClient/PsnClient.csproj` (+1 -1) 📝 `CompatBot/CompatBot.csproj` (+12 -12) 📝 `CompatBot/EventHandlers/LogParsing/LogParser.LogSections.cs` (+15 -13) 📝 `CompatBot/EventHandlers/LogParsing/LogParser.PipeReader.cs` (+4 -1) 📝 `CompatBot/EventHandlers/LogParsing/LogParser.StateMachineGenerator.cs` (+1 -6) 📝 `CompatBot/EventHandlers/LogParsing/POCOs/LogParseState.cs` (+3 -0) 📝 `CompatBot/EventHandlers/UnknownCommandHandler.cs` (+2 -1) ➕ `CompatBot/Utils/NameUniqueObjectCollection.cs` (+80 -0) 📝 `CompatBot/Utils/ResultFormatters/LogParserResultFormatter.GeneralNotesSection.cs` (+30 -29) 📝 `CompatBot/Utils/ResultFormatters/LogParserResultFormatter.WeirdSettingsSection.cs` (+16 -15) 📝 `CompatBot/Utils/ResultFormatters/LogParserResultFormatter.cs` (+26 -26) ➕ `CompatBot/Utils/UniqueList.cs` (+99 -0) </details> ### 📄 Description This will reduce memory footprint and speed up processing in checks --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-15 15:56:23 -05:00
yindo closed this issue 2026-02-15 15:56:23 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Hacktoberfest
Hacktoberfest
bug
enhancement
enhancement
enhancement
fun stuff
good first issue
good first issue
help wanted
high priority
invalid
log analyzer
pull-request
Mirrored from GitHub Pull Request
 questionable
questionable
wontfix
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: RPCS3/discord-bot#595
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?