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

Parameter passthrough #183

Merged
merged 3 commits into from
Sep 25, 2019
Merged

Parameter passthrough #183

merged 3 commits into from
Sep 25, 2019

Conversation

vodorok
Copy link
Collaborator

@vodorok vodorok commented Sep 19, 2019

There is a new feature of CodeChecker configuration. Now there is a
direct parameter passthrough to the CodeChecker package. It can be
configured in the preferences page.

There is also a live display of the analyze command right below the
passthrough specification Text widget. Uneditable, but can be copied
from. With this, an actual analyze run is more easily reproducible.

Resolves #104

There is a new feature of CodeChecker configuration. Now there is a
direct parameter passthrough to the CodeChecker package. It can be
configured in the preferences page.

There is also a live display of the analyze command right below the
passthrough specification Text widget. Uneditable, but can be copied
from. With this, an actual analyze run is more easily reproducible.
@vodorok vodorok requested review from gamesh411 and dkrupp September 19, 2019 15:13
@vodorok
Copy link
Collaborator Author

vodorok commented Sep 19, 2019

Fist part of pr. A second commit is coming that removes the existing checker selector.

@Ericsson Ericsson deleted a comment Sep 23, 2019
@Ericsson Ericsson deleted a comment Sep 23, 2019
@Ericsson Ericsson deleted a comment Sep 23, 2019
@Ericsson Ericsson deleted a comment Sep 23, 2019
@Ericsson Ericsson deleted a comment Sep 23, 2019
@Ericsson Ericsson deleted a comment Sep 23, 2019
Copy link
Collaborator

@gamesh411 gamesh411 left a comment

Choose a reason for hiding this comment

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

It is nice to see things simplify.

@gamesh411 gamesh411 merged commit 52d6141 into Ericsson:master Sep 25, 2019
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.

CodeChecker check passthrough parameter
2 participants