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.

Sign up for GitHub

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

Jump to bottom

Integrate CodeChecker log into executor #108

Merged
merged 6 commits into from
May 20, 2022
Merged

Integrate CodeChecker log into executor #108

merged 6 commits into from
May 20, 2022

Conversation

Discookie
Copy link
Collaborator

Fixes #38.

Instead of running CodeChecker log as a separate process, it is now integrated into the `backend/executor.

image

  • Added commands, build tasks, UI buttons for running CodeChecker log.
  • Added settings to specify custom args as well as a build command, which can be overridden by commands or tasks.
  • Added a couple tests for the backend.

@Discookie Discookie added enhancement New feature or request refactoring labels Apr 29, 2022
@Discookie Discookie added this to the 1.4.0 milestone Apr 29, 2022
@dkrupp dkrupp requested a review from vodorok May 6, 2022 09:19
vodorok
vodorok reviewed May 20, 2022
View reviewed changes
Copy link
Collaborator

@vodorok vodorok left a comment •
edited
Loading

Choose a reason for hiding this comment

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

.

vodorok
vodorok approved these changes May 20, 2022
View reviewed changes
Copy link
Collaborator

@vodorok vodorok left a comment

Choose a reason for hiding this comment

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

Looks good to me!

vodorok
vodorok approved these changes May 20, 2022
View reviewed changes
Copy link
Collaborator

@vodorok vodorok left a comment

Choose a reason for hiding this comment

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

Looks good to me!

vodorok
vodorok approved these changes May 20, 2022
View reviewed changes
Copy link
Collaborator

@vodorok vodorok left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@Discookie Discookie merged commit e39772c into Ericsson:main May 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vodorok vodorok vodorok approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request refactoring
Projects
None yet
Milestone
1.4.0
Development

Successfully merging this pull request may close these issues.

Configurable options for CodeChecker log command
2 participants
@Discookie @vodorok