Skip to content

Remove parameter from console.groupEnd()#3517

Merged
captainbrosset merged 3 commits intoMicrosoftDocs:mainfrom
jzk-onshift:groupEnd-param-fix
Jul 29, 2025
Merged

Remove parameter from console.groupEnd()#3517
captainbrosset merged 3 commits intoMicrosoftDocs:mainfrom
jzk-onshift:groupEnd-param-fix

Conversation

@jzk-onshift
Copy link
Copy Markdown
Contributor

@jzk-onshift jzk-onshift commented Jul 24, 2025

Rendered article sections for review:

PR background:
Edge documentation included a parameter for groupEnd, but groupEnd does not accept parameters.
See MDN.

AB#58527965

@jzk-onshift
Copy link
Copy Markdown
Contributor Author

@jzk-onshift please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@microsoft-github-policy-service agree [company="{your company}"]

Options:

  • (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
  • (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree company="Shiftkey, LLC"

@mikehoffms mikehoffms added the cat: devtools DevTools-related content. label Jul 25, 2025
Comment thread microsoft-edge/devtools/console/index.md Outdated
@learn-build-service-prod
Copy link
Copy Markdown

Learn Build status updates of commit 09966ba:

✅ Validation status: passed

File Status Preview URL Details
microsoft-edge/devtools/console/api.md ✅Succeeded View
microsoft-edge/devtools/console/console-log.md ✅Succeeded View
microsoft-edge/devtools/console/index.md ✅Succeeded View

For more details, please refer to the build report.

@mikehoffms mikehoffms changed the title Remove parameter from groupEnd Remove parameter from console.groupEnd() Jul 25, 2025
Comment thread microsoft-edge/devtools/console/console-log.md
@jzk-onshift
Copy link
Copy Markdown
Contributor Author

jzk-onshift commented Jul 25, 2025

@mikehoffms I noticed those typos while I was in there, but there was a note that mentioned they were included intentionally. I thought that was curious.

@mikehoffms mikehoffms self-requested a review July 25, 2025 16:32
Copy link
Copy Markdown
Collaborator

@mikehoffms mikehoffms left a comment

Choose a reason for hiding this comment

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

LGTM - Writer/Editor

@captainbrosset captainbrosset merged commit a3e4fa2 into MicrosoftDocs:main Jul 29, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cat: devtools DevTools-related content.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants