Skip to content

Conversation

CJSmith-0141
Copy link
Contributor

Understandably the IntelliJ section was removed. This portion of the documentation is still very useful, especially for people who are used to being able to hit the play button on individual tests. I also re-worded things slightly while removing references to the abandoned plugin.

docs/intellij.md Outdated

Weaver provides IntelliJ integration by means of a JUnit runner that IntelliJ picks up automatically.

## Ignorning individual tests
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I very much appreciate the added documentation, but this section can be moved elsewhere as it's not Intellij specific. We could add a link from here to whatever section we end up moving this to

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would a new page called Frequently Asked Questions be appropriate?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It very much would !

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, it's mostly the same. Reworded sections to be questions and tried to make it not IntelliJ specific.

Understandably the IntelliJ section was removed. This portion of the
documentation is still very useful, especially for people who are used
to being able to hit the play button on individual tests.
The old intellij sections has been moved to "Frequently Asked Questions".
@Baccata Baccata merged commit 9efa587 into typelevel:main Mar 14, 2024
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.

2 participants