[PR #5451] [CLOSED] docs: improve code comments and remove TODO items #5215

New issue

Closed

opened 2026-03-17 02:40:56 +03:00 by kerem · 0 comments

kerem commented

2026-03-17 02:40:56 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/hoppscotch/hoppscotch/pull/5451
Author: @shaiksajidhussain
Created: 10/5/2025
Status: ❌ Closed

Base: main ← Head: docs/improve-code-comments

📝 Commits (1)

b32d7df docs: improve code comments and remove TODO items

📊 Changes

3 files changed (+5 additions, -5 deletions)

View changed files

📝 packages/hoppscotch-common/src/components/smart/ChangeLanguage.vue (+1 -2)
📝 packages/hoppscotch-common/src/helpers/editor/extensions/HoppPredefinedVariables.ts (+3 -2)
📝 packages/hoppscotch-data/src/predefinedVariables.ts (+1 -1)

📄 Description

Replace vague TODO comment in ChangeLanguage component with clear description
Remove unnecessary TODO in changeLocale function (already implemented)
Improve comment in HoppPredefinedVariables.ts explaining alternative implementation
Update comment in predefinedVariables.ts for better extensibility documentation

Improves code maintainability and developer experience

Closes #

What's changed

Notes to reviewers

_{🔄 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/hoppscotch/hoppscotch/pull/5451 **Author:** [@shaiksajidhussain](https://github.com/shaiksajidhussain) **Created:** 10/5/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `docs/improve-code-comments` --- ### 📝 Commits (1) - [`b32d7df`](https://github.com/hoppscotch/hoppscotch/commit/b32d7dfecb180fecf44bb2233f865e7d693c2c76) docs: improve code comments and remove TODO items ### 📊 Changes **3 files changed** (+5 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `packages/hoppscotch-common/src/components/smart/ChangeLanguage.vue` (+1 -2) 📝 `packages/hoppscotch-common/src/helpers/editor/extensions/HoppPredefinedVariables.ts` (+3 -2) 📝 `packages/hoppscotch-data/src/predefinedVariables.ts` (+1 -1) </details> ### 📄 Description - Replace vague TODO comment in ChangeLanguage component with clear description - Remove unnecessary TODO in changeLocale function (already implemented) - Improve comment in HoppPredefinedVariables.ts explaining alternative implementation - Update comment in predefinedVariables.ts for better extensibility documentation Improves code maintainability and developer experience   Closes #   ### What's changed   ### Notes to reviewers  --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>