Skip to content

Plugin(apps::protocols::snmp) - Mode(collection) : Extend automated tests #5749

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
Open
sfarouq-ext wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Plugin(apps::protocols::snmp) - Mode(collection) : Extend automated tests #5749

sfarouq-ext wants to merge 2 commits into from

Conversation

@sfarouq-ext
Copy link
Contributor

@sfarouq-ext sfarouq-ext commented Sep 15, 2025 •
edited
Loading

Refs: 1380

Checklist

  • I have followed the coding style guidelines provided by Centreon
  • I have commented my code, especially hard-to-understand areas of the PR.
  • I have rebased my development branch on the base branch (develop).
  • In case of a new plugin, I have created the new packaging directory accordingly.
  • I have implemented automated tests related to my commits.
    • Data used for automated tests are anonymized.
  • I have reviewed all the help messages in all the .pm files I have modified.
    • All sentences begin with a capital letter.
    • All sentences end with a period.
    • I am able to understand all the help messages, if not, exchange with the PO or TW to rewrite them.
  • After having created the PR, I will make sure that all the tests provided in this PR have run and passed.

@sfarouq-ext sfarouq-ext requested a review from a team as a code owner September 15, 2025 08:37
@github-actions
Copy link

github-actions bot commented Sep 15, 2025

Logo
Checkmarx One – Scan Summary & Details – 00611fe4-ce49-44df-a24d-45855a0273a1

Great job! No new security vulnerabilities introduced in this pull request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sechkem sechkem Awaiting requested review from sechkem sechkem is a code owner automatically assigned from centreon/owners-robot-e2e

@scresto31 scresto31 Awaiting requested review from scresto31 scresto31 is a code owner automatically assigned from centreon/owners-robot-e2e

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

skip-workflow-plugins-analysis

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sfarouq-ext