Skip to content

msedge: remove '=' from long options #16872

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

msedge: remove '=' from long options #16872

wants to merge 1 commit into from
+4 −4

Conversation

AbhisekLimbu
Copy link

  • [x ] The page(s) are in the correct platform directories: common, linux, osx, windows, sunos, android, etc.
  • The page(s) have at most 8 examples.
  • The page description(s) have links to documentation or a homepage.
  • The page(s) follow the content guidelines.
  • The page(s) follow the style guide.
  • The PR title conforms to the recommended templates.
  • Version of the command being documented (if known):

@CLAassistant
Copy link

CLAassistant commented Jun 16, 2025 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added page edit Changes to an existing page(s). review needed Prioritized PRs marked for reviews from maintainers. labels Jun 16, 2025
@Managor Managor changed the title fix(windows): remove '=' from long options in msedge.md msedhe: remove '=' from long options Jun 17, 2025
@Managor Managor changed the title msedhe: remove '=' from long options msedge: remove '=' from long options Jun 17, 2025
@Managor
Copy link
Collaborator

Managor commented Jun 17, 2025

#16827

@AbhisekLimbu AbhisekLimbu force-pushed the fix/octo-remove-equals branch from 1da7984 to b61d5d8 Compare June 17, 2025 20:19
@Managor
Copy link
Collaborator

Managor commented Jun 17, 2025 •
edited
Loading

Please don't force push as it erases history. It's nice to know what changes were made after what comments. We will in the end just squash and merge this PR so it doesn't matter if the PR history looks messy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sebastiaanspeck sebastiaanspeck sebastiaanspeck approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
page edit Changes to an existing page(s). review needed Prioritized PRs marked for reviews from maintainers.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@AbhisekLimbu @CLAassistant @Managor @sebastiaanspeck