Skip to content

MWPW-206311: drop getValidatedMasLibsUrl JSDoc, no comments in utils - #6663

Merged
mokimo merged 1 commit into
vhargrave/lcp-blocks-only-pre-mepfrom
osahin/pr-6643-jsdoc-followup
Sep 7, 2026
Merged

MWPW-206311: drop getValidatedMasLibsUrl JSDoc, no comments in utils#6663
mokimo merged 1 commit into
vhargrave/lcp-blocks-only-pre-mepfrom
osahin/pr-6643-jsdoc-followup

Conversation

@mokimo

@mokimo mokimo commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #6662 - zagi25's JSDoc note on #6643. Rather than convert it to a plain
comment, dropping it entirely: libs/utils/utils.js carries no comments by
convention, so the JSDoc (pre-existing, relocated verbatim from elsewhere in the
file) comes out with no replacement.

215/215 Utils tests passing, ESLint clean.

🤖 Generated with Claude Code

zagi25 flagged this JSDoc as inconsistent with the rest of the PR (#6643).
It was pre-existing code being relocated, not new authorship, but per repo
convention libs/utils/utils.js carries no comments - dropping it outright
rather than converting to a plain comment.
@aem-code-sync

aem-code-sync Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-sync branch
Commits

@mokimo
mokimo merged commit 0fa992a into vhargrave/lcp-blocks-only-pre-mep Sep 7, 2026
8 checks passed
@mokimo
mokimo deleted the osahin/pr-6643-jsdoc-followup branch September 7, 2026 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant