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

Add weggli to the linter for c #3285

Open
Pierre-Gronau-ndaal opened this issue Jan 14, 2024 · 8 comments
Open

Add weggli to the linter for c #3285

Pierre-Gronau-ndaal opened this issue Jan 14, 2024 · 8 comments
Labels
enhancement New feature or request

Comments

@Pierre-Gronau-ndaal
Copy link

Pierre-Gronau-ndaal commented Jan 14, 2024

Is your feature request related to a problem?
nope

Describe the solution you'd like
Enrich security detection capabilities

Describe alternatives you've considered
semgrep

Additional context

@Pierre-Gronau-ndaal Pierre-Gronau-ndaal added the enhancement New feature or request label Jan 14, 2024
@nvuillam
Copy link
Member

@Pierre-Gronau-ndaal semgrep is already integrated within MegaLinter, but activated only when a semgrep rules config var is defined in the analyzed repository

Example: REPOSITORY_SEMGREP_RULESETS: ["p/docker-compose","p/owasp-top-ten"]

Are there SemGrep rulesets similar to weggli rules ?

@Pierre-Gronau-ndaal
Copy link
Author

In principle and for sure there is a overlapping

Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity.
It will be closed in 14 days if no further activity occurs.
Thank you for your contributions.

If you think this issue should stay open, please remove the O: stale 🤖 label or comment on the issue.

@github-actions github-actions bot added the O: stale 🤖 This issue or pull request is stale, it will be closed if there is no activity label Feb 20, 2024
@nvuillam
Copy link
Member

nvuillam commented Mar 2, 2024

@Pierre-Gronau-ndaal > weggli seems dedicated to c & c++ so has probably more capabilities than SemGrep, & probably better performances

Would you like to add it in a PR ? :)

@Pierre-Gronau-ndaal
Copy link
Author

I will do a PR If it is wished

@github-actions github-actions bot removed the O: stale 🤖 This issue or pull request is stale, it will be closed if there is no activity label Mar 3, 2024
Copy link
Contributor

github-actions bot commented Apr 2, 2024

This issue has been automatically marked as stale because it has not had recent activity.
It will be closed in 14 days if no further activity occurs.
Thank you for your contributions.

If you think this issue should stay open, please remove the O: stale 🤖 label or comment on the issue.

@github-actions github-actions bot added the O: stale 🤖 This issue or pull request is stale, it will be closed if there is no activity label Apr 2, 2024
@nvuillam nvuillam removed the O: stale 🤖 This issue or pull request is stale, it will be closed if there is no activity label Apr 2, 2024
Copy link
Contributor

github-actions bot commented May 3, 2024

This issue has been automatically marked as stale because it has not had recent activity.
It will be closed in 14 days if no further activity occurs.
Thank you for your contributions.

If you think this issue should stay open, please remove the O: stale 🤖 label or comment on the issue.

@github-actions github-actions bot added the O: stale 🤖 This issue or pull request is stale, it will be closed if there is no activity label May 3, 2024
@nvuillam
Copy link
Member

nvuillam commented May 5, 2024

@Pierre-Gronau-ndaal do you still wish to make the PR ? :)

@nvuillam nvuillam removed the O: stale 🤖 This issue or pull request is stale, it will be closed if there is no activity label May 5, 2024
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

No branches or pull requests

2 participants