Skip to content
This repository was archived by the owner on Dec 13, 2022. It is now read-only.

enh(ldap): improve ldap codebase readability #11216

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from
Open

enh(ldap): improve ldap codebase readability #11216

wants to merge 8 commits into from

Conversation

kduret
Copy link
Contributor

@kduret kduret commented Jun 9, 2022

Description

improve ldap codebase readability

Fixes MON-13915

Type of change

  • Patch fixing an issue (non-breaking change)
  • New functionality (non-breaking change)
  • Breaking change (patch or feature) that might cause side effects breaking part of the Software

Target serie

  • 21.04.x
  • 21.10.x
  • 22.04.x
  • 22.10.x (master)

@sonarqube-decoration
Copy link

SonarQube Quality Gate

Quality Gate passed

Bug B 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 315 Code Smells

0.0% 0.0% Coverage
5.3% 5.3% Duplication

@sonarqube-decoration
Copy link

SonarQube Quality Gate

Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell B 5 Code Smells

No Coverage information No Coverage information
10.2% 10.2% Duplication

@emabassi-ext emabassi-ext marked this pull request as ready for review November 4, 2022 14:14
@emabassi-ext emabassi-ext requested review from a team, TamazC and wtermellil November 4, 2022 14:14
@sonarqube-decoration
Copy link

SonarQube Quality Gate

Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell B 5 Code Smells

No Coverage information No Coverage information
10.0% 10.0% Duplication

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

@TamazC TamazC Awaiting requested review from TamazC

@wtermellil wtermellil Awaiting requested review from wtermellil

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
area/backend
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kduret @emabassi-ext