Skip to content

Fix spelling in AD provider code comments#8565

Merged
danlavu merged 1 commit into
SSSD:masterfrom
striker314:spelling-updates
Apr 10, 2026
Merged

Fix spelling in AD provider code comments#8565
danlavu merged 1 commit into
SSSD:masterfrom
striker314:spelling-updates

Conversation

@striker314

Copy link
Copy Markdown
Contributor

Fixed some simple spelling errors within the AD providers code comments.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request corrects several typographical errors in comments and log messages across various files in the AD provider, including ad_common.c, ad_common.h, ad_gpo.c, ad_gpo_child.c, ad_gpo_child_utils.c, ad_id.c, and ad_subdomains.c. I have no feedback to provide.

@sumit-bose sumit-bose added the Trivial A single reviewer is sufficient to review the Pull Request label Apr 2, 2026
@sumit-bose
sumit-bose requested a review from danlavu April 2, 2026 12:43
@sssd-bot

sssd-bot commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

The pull request was accepted by @danlavu with the following PR CI status:


🟢 osh-diff-scan:fedora-rawhide-x86_64:upstream (success)
🟢 rpm-build:centos-stream-10-x86_64:upstream (success)
🟢 rpm-build:fedora-42-x86_64:upstream (success)
🟢 rpm-build:fedora-43-x86_64:upstream (success)
🟢 rpm-build:fedora-44-x86_64:upstream (success)
🟢 rpm-build:fedora-rawhide-x86_64:upstream (success)
🟢 Analyze (target) / cppcheck (success)
🟡 Build / freebsd (in_progress)
🟡 Build / make-distcheck (in_progress)
🟡 ci / intgcheck (centos-10) (in_progress)
🟡 ci / intgcheck (fedora-42) (in_progress)
🟡 ci / intgcheck (fedora-43) (in_progress)
🟡 ci / intgcheck (fedora-44) (in_progress)
🟡 ci / intgcheck (fedora-45) (in_progress)
🟢 ci / prepare (success)
🟡 ci / system (centos-10) (in_progress)
🟡 ci / system (fedora-42) (in_progress)
🟡 ci / system (fedora-43) (in_progress)
🟡 ci / system (fedora-44) (in_progress)
🟡 ci / system (fedora-45) (in_progress)
➖ Coverity scan / coverity (skipped)
🟡 Static code analysis / codeql (in_progress)
🟢 Static code analysis / pre-commit (success)
🟢 Static code analysis / python-system-tests (success)


There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging.

@danlavu
danlavu merged commit 58cc4d2 into SSSD:master Apr 10, 2026
16 checks passed
@striker314
striker314 deleted the spelling-updates branch April 10, 2026 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Accepted backport-to-sssd-2-9 Trivial A single reviewer is sufficient to review the Pull Request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants