Skip to content

Fix spelling in AD provider code comments#8565

Open
striker314 wants to merge 1 commit intoSSSD:masterfrom
striker314:spelling-updates
Open

Fix spelling in AD provider code comments#8565
striker314 wants to merge 1 commit intoSSSD:masterfrom
striker314:spelling-updates

Conversation

@striker314
Copy link
Copy Markdown

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

Copy link
Copy Markdown

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

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
Reviewed-by: Dan Lavu <dlavu@redhat.com>
@sssd-bot
Copy link
Copy Markdown
Contributor

sssd-bot commented Apr 9, 2026

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.

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