Skip to content

Implement store_codechecker rule#216

Closed
furtib wants to merge 5 commits into
Ericsson:mainfrom
furtib:codechecker_store
Closed

Implement store_codechecker rule#216
furtib wants to merge 5 commits into
Ericsson:mainfrom
furtib:codechecker_store

Conversation

@furtib

@furtib furtib commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

Why:
Users wanted an easy way to store

What:

  • Added custom rule for storing results of codechecker analysis.
  • Added test for the new custom rule.

Addresses:
#107

@furtib furtib requested a review from Szelethus April 7, 2026 15:03
@furtib furtib self-assigned this Apr 7, 2026
@furtib furtib added the enhancement New feature or request label Apr 7, 2026

@nettle nettle left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

As I mentioned before I will reject any changes related to CodeChecker store implementation until the design is presented and approved. Please contact me for details

@furtib furtib marked this pull request as draft April 9, 2026 14:31
@furtib furtib force-pushed the codechecker_store branch from 45f0bf9 to ad7f04f Compare June 16, 2026 09:55
@furtib furtib force-pushed the codechecker_store branch from 7d9532d to 27da73d Compare June 16, 2026 10:14

@nettle nettle left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please explore use-cases and existing implementations before suggesting new one.
Let's discuss this offline

@furtib

furtib commented Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

Sorry, I meant to do offline exploration; this PR should have been closed.

@furtib furtib closed this Jun 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants