Skip to content

feat: support jumping with g+G in incident views - #426

Merged
clcollins merged 3 commits into
clcollins:mainfrom
holysoles:jump_in_incident_views
Aug 10, 2026
Merged

feat: support jumping with g+G in incident views #426
clcollins merged 3 commits into
clcollins:mainfrom
holysoles:jump_in_incident_views

Conversation

@holysoles

Copy link
Copy Markdown
Contributor

What type of PR is this?

(bug/feature/cleanup/documentation)

Feature

What this PR does / Why we need it?

This PR extends the vim-style jump motions (g/G) to the tabs within the incident viewer. Previously these keystrokes were ignored. This allows quickly scrolling SLs and other incident contextual information.

This PR also fixes the getlint Makefile recipe to check for golangci-lint the same way it is invoked in lint. Previously, having golangci-lint installed to a location like /usr/bin would fail to run lint.

Which Jira/Github issue(s) does this PR fix?

Resolves #

N/A

Special notes for your reviewer

Pre-checks (if applicable)

  • Ran unit tests locally against the changes
  • Included documentation changes with PR
    • existing documentation covers this

@clcollins clcollins added skip-readme Skip README update CI check skip-quickstart Bypass quickstart-check when keybinding file changes have no quickstart impact skip-plan Skip plan document CI check labels Aug 10, 2026
@clcollins
clcollins merged commit 20cf81c into clcollins:main Aug 10, 2026
11 of 12 checks passed
clcollins added a commit that referenced this pull request Aug 11, 2026
PR #426 (g/G navigation in incident viewer) was silently reverted when
PR #419's rebase resolved a conflict by keeping only the #419 side.
CI could not catch it because the feature and its test were removed
together. This restores all three changes verbatim from PR #426.

Co-authored-by: agent-bot <agent-bot@localhost>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-plan Skip plan document CI check skip-quickstart Bypass quickstart-check when keybinding file changes have no quickstart impact skip-readme Skip README update CI check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants