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

Clarify initial setup steps #58

Open
firewave opened this issue Dec 23, 2020 · 2 comments
Open

Clarify initial setup steps #58

firewave opened this issue Dec 23, 2020 · 2 comments

Comments

@firewave
Copy link
Collaborator

The plugin has just an average rating at https://plugins.jetbrains.com/plugin/8143-cppcheck/reviews.

It seems most of them stem from the documentation for the initial setup being confusing. We need to figure out what confuses people and improve the documentation accordingly. Maybe even rewrite it from scratch.

@johnthagen
Copy link
Owner

Agreed. For what it's worth, most of the poor ratings came before I made the landing page a lot more clear. But overall, if the plugin better guided a user through what they needed to do (such that you didn't really need to refer to docs) that would probably be best.

When I initially wrote this plugin I was new to IntelliJ plugins and wrote this one based on an existing Cpplint plugin that followed a similar usage flow. I think this workflow could definitely be improved with some more digging into what IntelliJ offers (years later).

@firewave
Copy link
Collaborator Author

I am planning on making the configuration regarding the --language= option a bit easier for 1.6.0 by handling it internally.

The enhanced error balloon should probably also help in some situations.

I also have a document ready which covers most of the known issues in detail as well as offering some workaround. That should hopefully resolve some user issues as well. While I was writing that I actually thought of lots of things which resulted in those past few issues I added.

I am still not sure how to integrate it with the plugin description for the CLion repository though.

firewave added a commit to firewave/clion-cppcheck that referenced this issue Mar 28, 2023
firewave added a commit to firewave/clion-cppcheck that referenced this issue Mar 28, 2023
firewave added a commit to firewave/clion-cppcheck that referenced this issue Mar 28, 2023
firewave added a commit that referenced this issue Mar 29, 2023
* README.md: documented `Show Cppcheck XML Output` action and its limitations

* README.md: re-wrote "Known Issues" section

* some minor documentation improvements

* README.md: use proper capitalization for `Cppcheck` and do not quote it

* README.md: rewrote `Installation` section - refs #58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants