Skip to content

Conversation

@ElizabethSamuel-MSFT
Copy link
Contributor

No description provided.

});
}

async function getSpecificGeometricShapes() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Consider having two addShape methods so the getFirst and getSpecific methods return different things.


if (geometricShape.isNullObject) {
console.log("No geometric shapes found in the document body.");
return;
Copy link
Collaborator

Choose a reason for hiding this comment

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

This return seems unnecessary.


if (heptagon.isNullObject) {
console.log("No heptagons found in the document body.");
return;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same for this return - consider removing it.

@AlexJerabek
Copy link
Collaborator

Approving with a few suggestions.

@ElizabethSamuel-MSFT ElizabethSamuel-MSFT merged commit 0679d3a into main Oct 31, 2025
1 check passed
@ElizabethSamuel-MSFT ElizabethSamuel-MSFT deleted the eliz-word-geometric-shapes branch October 31, 2025 20:56
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.

3 participants