Skip to content

Remove GetOptionsDiagnostics, global diags in editor, restore checker affinity#3285

Draft
jakebailey wants to merge 23 commits intomainfrom
jabaile/restore-per-checker-looping
Draft

Remove GetOptionsDiagnostics, global diags in editor, restore checker affinity#3285
jakebailey wants to merge 23 commits intomainfrom
jabaile/restore-per-checker-looping

Conversation

@jakebailey
Copy link
Copy Markdown
Member

This builds on #2322. Probably could replace it and just go for this.

From #2322:

  • Remove GetOptionsDiagnostics
  • Expose global diags to the editor.

Then:

  • Restore the looping over checkers to walk files in a consistent order and keep them in the same goroutine during many file checking.
  • Refactor the CheckerPool interface again, now down to just one single function.
    • As a side effect, fixes a bug where the project pool was not giving requests a consistent checker.

@jakebailey jakebailey changed the title Remove GetOptionsDiagnostics, global diags in checker, restore checker affinity Remove GetOptionsDiagnostics, global diags in editor, restore checker affinity Mar 29, 2026
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