Skip to content

docs: fix 'recieved' -> 'received' typo in IAutoCredentials Javadoc#8529

Merged
rzo1 merged 1 commit into
apache:masterfrom
SAY-5:fix-autocredentials-recieved-typo
Apr 17, 2026
Merged

docs: fix 'recieved' -> 'received' typo in IAutoCredentials Javadoc#8529
rzo1 merged 1 commit into
apache:masterfrom
SAY-5:fix-autocredentials-recieved-typo

Conversation

@SAY-5

@SAY-5 SAY-5 commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

Javadoc-only typo fix in storm-client/src/jvm/org/apache/storm/security/auth/IAutoCredentials.java line 42:
credentials are recievedcredentials are received.

No code change, no behavior change — this is only the English word spelling in a single Javadoc sentence. CI skipping this is expected for doc-only changes.

Javadoc in storm-client/src/jvm/org/apache/storm/security/auth/IAutoCredentials.java line 42 read 'credentials are recieved'. Fixed to 'received'. Comment-only change.

Signed-off-by: SAY-5 <SAY-5@users.noreply.github.com>
@SAY-5

SAY-5 commented Apr 16, 2026

Copy link
Copy Markdown
Contributor Author

Friendly ping — CI is green and the fix is approved. Mind merging when convenient?

@rzo1

rzo1 commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

CI didn't run and this is a doc only change. The PR title and commit msg are misleading. Did you forget some content?

@SAY-5 SAY-5 changed the title security: fix 'recieved' -> 'received' in IAutoCredentials Javadoc docs: fix 'recieved' -> 'received' typo in IAutoCredentials Javadoc Apr 17, 2026
@SAY-5

SAY-5 commented Apr 17, 2026

Copy link
Copy Markdown
Contributor Author

@rzo1 thanks for the review — you're right the security: prefix was misleading since this is purely a Javadoc typo fix (no security impact). Retitled to docs: and clarified the description. The PR content is just the single Javadoc word change you see in the diff; no other changes. CI appears to skip doc-only changes intentionally.

@rzo1 rzo1 merged commit c09f03a into apache:master Apr 17, 2026
@rzo1

rzo1 commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

Thx @SAY-5

@reiabreu reiabreu added this to the 2.8.7 milestone Apr 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants