8000 IntelliJ GitHub Pull Request Integration looks awful with code experts check · Issue #481 · linear-b/gitstream · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
IntelliJ GitHub Pull Request Integration looks awful with 8000 code experts check #481
Open
@tresat

Description

@tresat

Describe the bug

The newest version of IntelliJ has support for opening GitHub PRs in the IDE, see: https://blog.jetbrains.com/idea/2024/02/intellij-idea-2024-1-eap-6/#in-editor-code-review and https://www.jetbrains.com/help/idea/work-with-github-pull-requests.html#provide-feedback.

This also shows the PR's comments directly in the editor. This feature does not play nicely with the code experts check.

To Reproduce

Steps to reproduce the behavior:

  1. Open a PR in IntelliJ that has gitStream enabled and has received a code experts comment.
  2. An example PR: Decouple version conflict error handling gradle/gradle#28653
  3. In the IDE, the code experts comment is rendered as a gigantic several-"page" document that gets in the way of other more relevant human comments on the PR, to the point the people unfamiliar with the system might not even realize those other comments are visible here.

Expected behavior

This comment should be condensed somehow. Even better, it could be rendered as a collapsable HTML table in the IDE.

Screenshots
image
image
image

Additional context

Very likely this is nothing specific to gitStream. It will probably have to be addressed on the IntelliJ side. However, the way the month abbreviations ("APR", "MAR") are split across multiple lines makes me wonder if something could be done in gitStream (perhaps with more HTML?) to make the table at least look usable within the IDE.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      0