Skip to content

fix: rename getLocallocalInfo to getLocalInfo (typo)#333

Open
machiabeli wants to merge 1 commit intoblackboardsh:mainfrom
machiabeli:fix/rename-getlocalinfo-typo
Open

fix: rename getLocallocalInfo to getLocalInfo (typo)#333
machiabeli wants to merge 1 commit intoblackboardsh:mainfrom
machiabeli:fix/rename-getlocalinfo-typo

Conversation

@machiabeli
Copy link
Copy Markdown

�[38;5;238m─────┬──────────────────────────────────────────────────────────────────────────�[0m
�[38;5;238m 1�[0m �[38;5;238m│�[0m �[38;2;255;255;255m## Summary�[0m
�[38;5;238m 2�[0m �[38;5;238m│�[0m
�[38;5;238m 3�[0m �[38;5;238m│�[0m �[38;2;255;255;255m- Renames getLocallocalInfo (double "local" typo) to getLocalInfo to match the documented API name�[0m
�[38;5;238m 4�[0m �[38;5;238m│�[0m �[38;2;255;255;255m- Updates all 10 call sites: 9 in package/src/bun/core/Updater.ts + 1 in kitchen/src/bun/index.ts�[0m
�[38;5;238m 5�[0m �[38;5;238m│�[0m �[38;2;255;255;255m- This is a breaking change for anyone using the typo'd name directly, but aligns with what the docs already say�[0m
�[38;5;238m 6�[0m �[38;5;238m│�[0m
�[38;5;238m 7�[0m �[38;5;238m│�[0m �[38;2;255;255;255mFixes #254�[0m
�[38;5;238m─────┴──────────────────────────────────────────────────────────────────────────�[0m

…──────────────────────�[0m

�[38;5;238m   1�[0m �[38;5;238m│�[0m �[38;2;255;255;255mfix: rename getLocallocalInfo to getLocalInfo (typo)�[0m
�[38;5;238m   2�[0m �[38;5;238m│�[0m
�[38;5;238m   3�[0m �[38;5;238m│�[0m �[38;2;255;255;255mThe Updater method was named getLocallocalInfo (double "local") but�[0m
�[38;5;238m   4�[0m �[38;5;238m│�[0m �[38;2;255;255;255mdocumented as getLocalInfo. Renames all 10 call sites across�[0m
�[38;5;238m   5�[0m �[38;5;238m│�[0m �[38;2;255;255;255mUpdater.ts and the kitchen example to match the documented API.�[0m
�[38;5;238m   6�[0m �[38;5;238m│�[0m
�[38;5;238m   7�[0m �[38;5;238m│�[0m �[38;2;255;255;255mFixes blackboardsh#254�[0m
�[38;5;238m─────┴──────────────────────────────────────────────────────────────────────────�[0m
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.

1 participant