-
Notifications
You must be signed in to change notification settings - Fork 686
Implement CodeLens for TODO comments to improve action visibility #7954
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
Conversation
Co-authored-by: joshspicer <23246594+joshspicer@users.noreply.github.com>
Co-authored-by: joshspicer <23246594+joshspicer@users.noreply.github.com>
Co-authored-by: joshspicer <23246594+joshspicer@users.noreply.github.com>
Co-authored-by: joshspicer <23246594+joshspicer@users.noreply.github.com>
|
@copilot this works great. Add a setting (on by default) to disable these code lens |
|
@copilot remove the d.ts file changes! |
Added the setting
Added the setting |
4a30078 to
98b217c
Compare
…tion Co-authored-by: joshspicer <23246594+joshspicer@users.noreply.github.com>
Summary: Implement CodeLens for TODO Comments ✅
This PR successfully implements a CodeLens provider to make TODO comment actions more visible to users by displaying them directly above the code lines.
Implementation Checklist:
What Changed:
Before this PR:
After this PR:
Recent Updates:
lineNumberparameter fromfindTodoInLinemethodcacheAndReturntosetCachedResultfor better clarityFiles Changed:
src/issues/issueTodoProvider.ts
src/github/copilotRemoteAgent.ts
src/issues/issueFeatureRegistrar.ts
src/common/settingKeys.ts
CREATE_ISSUE_TRIGGERS_SHOW_CODE_LENSconstantpackage.json
githubIssues.createIssueTriggersShowCodeLensboolean settingpackage.nls.json
src/test/issues/issueTodoProvider.test.ts
documentation/IssueFeatures.md
Quality Assurance:
✅ All existing tests pass
✅ ESLint: No warnings or errors
✅ Hygiene checks: Pass
✅ TypeScript compilation: Success
✅ Code review feedback addressed
✅ Setting properly integrated with VS Code configuration system
✅ Documentation updated
Status: ✅ Ready for review and merge
Breaking Changes: None
Migration Required: None
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.