Skip to content
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

More precise flow step navigation #51

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

wwong
Copy link
Contributor

@wwong wwong commented Jan 19, 2024

Hacking some stuff together for my own use. I'll separate this out into two distinct PRs later - one for viewing by GHAS alert numbers and another for the step navigation changes.

User-facing changes:

wwong added 2 commits March 11, 2024 23:35
Kinda clunky. Ideally, we'd group the alerts by alert number range to prevent the vertical sprawl.
Flow steps can belong to different parts of the same line.
This commit makes selections more precise by including column information from the SARIF step data.
@wwong wwong force-pushed the ww/more-precise-flow-step-navigation branch from 15024f6 to 7f28832 Compare March 12, 2024 07:06
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