Update AddMember.java comments

[melaniedejong]

Includes changing "member" to "principal" and linking to principal identifiers page instead of specifying recommended format.

Description

Fixes #

Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.

Checklist

• I have followed Sample Format Guide
• pom.xml parent set to latest shared-configuration
• Appropriate changes to README are included in PR
• These samples need a new API enabled in testing projects to pass (let us know which ones)
• These samples need a new/updated env vars in testing projects set to pass (let us know which ones)
• Tests pass: mvn clean verify required
• Lint passes: mvn -P lint checkstyle:check required
• Static Analysis: mvn -P lint clean compile pmd:cpd-check spotbugs:check advisory only
• This sample adds a new sample directory, and I updated the CODEOWNERS file with the codeowners for this sample
• This sample adds a new Product API, and I updated the Blunderbuss issue/PR auto-assigner with the codeowners for this sample
• Please merge this PR for me once it is approved

[ghost]

Hi @melaniedejong and reviewers,

I'm currently reviewing this pull request and will post my full review in a few minutes. In the meantime, here's a quick summary to help everyone get up to speed:

This pull request updates the comments in AddMember.java to improve clarity and accuracy.

Intent:
The main goal is to replace the term "member" with "principal" throughout the code and comments for better alignment with Google Cloud IAM terminology. Additionally, instead of specifying a recommended format for principal identifiers, the updated comments now link to the official Google Cloud documentation page for principal identifiers.

Changes:

• The file iam/snippets/src/main/java/AddMember.java has been modified:
  • Lines 29-30: The comment // TODO: Replace with your member. has been changed to // TODO: Replace with your principal. // For examples, see https://cloud.google.com/iam/docs/principal-identifiers The placeholder member variable is updated to reflect the change.
  • Line 35: The comment // Adds a member to a pre-existing role. is updated to // Adds a principal to a pre-existing role.
  • Line 47: The comment // Update the policy to add the member. is updated to // Update the policy to add the principal.

These changes ensure consistency with Google Cloud IAM terminology and provide users with a direct link to relevant documentation.

---

Here's a little haiku to lighten the mood:

Code flows like a stream,
Clear comments guide the way,
Reviews bring the sun.

Let me know if you have any questions!

Best,
Gemini

[ghost]

Gemini encountered an error creating the review. You can try again by commenting @code-review-assist review.