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

Made it possible to override additional language options from the project settings #54

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JCash
Copy link

@JCash JCash commented May 1, 2016

Added display filters to ignore unwanted messages
Added a flag to control if the panel should hide when there are no messages

…ject settings

Added display filters to ignore unwanted messages
Added a flag to control if the panel should hide when there are no messages
@JCash
Copy link
Author

JCash commented May 1, 2016

Hi!
I really like this plugin, but I was having some difficulties using it so I thought I'd share my fixes.

First, I think having project specific setting overrides is good. That way you can have special settings for that special project (e.g I don't always want c++11)

Secondly, I just couldn't get rid of a certain warning (don't know why clang complained about it), so I thought of just filtering these out.

And lastly, since I work on a small laptop, it's beneficial to have the panel hide again, once I've fixed all the issues in the code.

These are the reasonings behind this pull request which I hope is beneficial to your project.

Thx,
Mathias

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