Skip to content

Fix typo: rename wago8chDioTestMachineMessageHndlerwago8chDioTestMachineMessageHandler#1352

Merged
ninelore merged 3 commits into
masterfrom
copilot/fix-spelling-in-handler-name
May 6, 2026
Merged

Fix typo: rename wago8chDioTestMachineMessageHndlerwago8chDioTestMachineMessageHandler#1352
ninelore merged 3 commits into
masterfrom
copilot/fix-spelling-in-handler-name

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 5, 2026

Corrects a misspelling in the exported function name (HndlerHandler) in the Wago 8-channel DIO test machine namespace.

Changes

  • Renamed wago8chDioTestMachineMessageHndlerwago8chDioTestMachineMessageHandler
  • Updated internal createEventHandler reference to use the corrected name

No other file in the codebase referenced the old misspelled name, so no backward-compatibility alias is needed.

Original prompt
Please apply the following diffs and create a pull request.
Once the PR is ready, give it a title based on the messages of the fixes being applied.

[{"message":"Corrected spelling of 'Hndler' to 'Handler' in function name.","fixFiles":[{"filePath":"electron/src/machines/minimal_machines/wago8chdiotestmachine/wago8chDioTestMachineNameSpace.ts","diff":"diff --git a/electron/src/machines/minimal_machines/wago8chdiotestmachine/wago8chDioTestMachineNameSpace.ts b/electron/src/machines/minimal_machines/wago8chdiotestmachine/wago8chDioTestMachineNameSpace.ts\n--- a/electron/src/machines/minimal_machines/wago8chdiotestmachine/wago8chDioTestMachineNameSpace.ts\n+++ b/electron/src/machines/minimal_machines/wago8chdiotestmachine/wago8chDioTestMachineNameSpace.ts\n@@ -36,7 +36,7 @@\n     }));\n \n // ========== Message Handler ==========\n-export function wago8chDioTestMachineMessageHndler(\n+export function wago8chDioTestMachineMessageHandler(\n   store: StoreApi<Wago8chDioTestMachineNamespaceStore>,\n   throttledUpdater: ThrottledStoreUpdater<Wago8chDioTestMachineNamespaceStore>,\n ): EventHandler {\n@@ -61,11 +61,14 @@\n   };\n }\n \n+export const wago8chDioTestMachineMessageHndler =\n+  wago8chDioTestMachineMessageHandler;\n+\n // ========== Namespace Hook ==========\n const useWago8chDioTestMachineImplementation =\n   createNamespaceHookImplementation<Wago8chDioTestMachineNamespaceStore>({\n     createStore: createWago8chDioTestMachineNamespaceStore,\n-    createEventHandler: wago8chDioTestMachineMessageHndler,\n+    createEventHandler: wago8chDioTestMachineMessageHandler,\n   });\n \n export function useWago8chDioTestMachineNamespace(\n"}]}]

Copilot AI changed the title [WIP] Fix spelling of 'Hndler' to 'Handler' in function name Fix typo: rename wago8chDioTestMachineMessageHndlerwago8chDioTestMachineMessageHandler May 5, 2026
Copilot AI requested a review from TheBest6337 May 5, 2026 15:34
@TheBest6337 TheBest6337 marked this pull request as ready for review May 5, 2026 15:39
@TheBest6337
Copy link
Copy Markdown
Member

@copilot check the electron frontend and backend and change the code so that the deprecation alias isnt needed / can be removed

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 5, 2026

@copilot check the electron frontend and backend and change the code so that the deprecation alias isnt needed / can be removed

Searched the entire codebase — no other file imports or uses the old misspelled name. The deprecation alias has been removed in commit 712aa04, leaving only the correctly-spelled wago8chDioTestMachineMessageHandler.

@TheBest6337 TheBest6337 requested a review from ninelore May 6, 2026 09:06
@ninelore ninelore merged commit a7fcf22 into master May 6, 2026
21 of 22 checks passed
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.

3 participants