-
-
Notifications
You must be signed in to change notification settings - Fork 549
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.
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
Add Names for Gigabyte B660M_DS3H_AX_DDR4 #1076
Add Names for Gigabyte B660M_DS3H_AX_DDR4 #1076
Conversation
priyanshuagrawal
commented
May 12, 2023
@@ -1618,6 +1618,31 @@ private static void GetIteConfigurationsB(ISuperIO superIO, Manufacturer manufac | |||
|
|||
break; | |||
|
|||
case Model.B660M_DS3H_AX_DDR4: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just going to comment on this line, but it applies to all names below. Please follow the naming of the rest in this file. Examples: Vcore instead of VCore. DIMM AB instead of DRAM CH(A/B). System Fan #1 instead of System 1, etc, etc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also you need to add parsing of B660M_DS3H_AX_DDR4 to this file: https://github.com/LibreHardwareMonitor/LibreHardwareMonitor/blob/master/LibreHardwareMonitorLib/Hardware/Motherboard/Identification.cs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there anything more i need to add?
PR LibreHardwareMonitor#1076 incorrectly introduced a build break due to wrong placement of parenthesis. Contributor doesn't appear to have compiled the code.
* Add Names for Gigabyte B660M_DS3H_AX_DDR4 * Updated naming, added parsing
…1079) PR LibreHardwareMonitor#1076 incorrectly introduced a build break due to wrong placement of parenthesis. Contributor doesn't appear to have compiled the code. Co-authored-by: ImLp <info@luisplacid.com>