Skip to content

Update piemenus.js#5830

Open
Aayush-kumar-ctrl wants to merge 1 commit intosugarlabs:masterfrom
Aayush-kumar-ctrl:patch-3
Open

Update piemenus.js#5830
Aayush-kumar-ctrl wants to merge 1 commit intosugarlabs:masterfrom
Aayush-kumar-ctrl:patch-3

Conversation

@Aayush-kumar-ctrl
Copy link

Update piemenus.js

docs(piemenus): add comprehensive JSDoc documentation

Added structured JSDoc comments to all major pie menu–related functions in js/piemenus.js. Documentation includes concise descriptions, parameter typing, and return annotations.

No functional changes were made.

Improves maintainability, developer clarity, and onboarding experience for contributors.

Update piemenus.js

docs(piemenus): add comprehensive JSDoc documentation

Added structured JSDoc comments to all major pie menu–related
functions in js/piemenus.js. Documentation includes concise
descriptions, parameter typing, and return annotations.

No functional changes were made.

Improves maintainability, developer clarity, and onboarding
experience for contributors.
@github-actions
Copy link
Contributor

✅ All Jest tests passed! This PR is ready to merge.

Copy link
Contributor

@vanshika2720 vanshika2720 left a comment

Choose a reason for hiding this comment

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

@Aayush-kumar-ctrl Thanks for the detailed JSDoc additions — this significantly improves readability and maintainability. The documentation is clear and well-structured, and the scope is clean (no functional changes).
LGTM.

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.

2 participants