Skip to content
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.

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

fix: remove stray $ character #53

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hugomatheuss
Copy link

fix: remove stray $ character

Removed a stray $ character that was causing issues when copying and pasting the code into the terminal.

Description

The commit removes a stray $ character from the code that was causing problems when copying and pasting the script into the terminal.

Motivation and context

This change is required because the stray $ character was causing an issue when users attempted to copy the code into the terminal. When copied directly, the $ symbol was interpreted incorrectly by the terminal.

How has this been tested?

I tested the change by:

Copying and pasting the code into a terminal after the $ character was removed.
Verifying that no errors occur during the paste operation and the code runs successfully.
Ensuring that the terminal accepts the pasted code without unexpected behavior.
Testing was done in a local environment on PhpStorm terminal.

Screenshots (if appropriate)

Capturar124

Checklist:

Go over all the following points before making your PR:

  • I have read the CONTRIBUTING document.
  • My pull request addresses exactly one patch/feature.
  • If my change requires a change to the documentation, I have updated it accordingly.

If you're unsure about any of these, don't hesitate to ask. We're here to help!

fix: remove stray `$` character

Removed a stray `$` character that was causing issues when copying and pasting the code into the terminal.
@tigitz
Copy link
Owner

tigitz commented Dec 30, 2024

I've given it some thought and concluded that it's fine. The $ symbol is traditionally used to indicate "as a user, not root," but in this context, it’s not worth disrupting the copy-paste functionality of GitHub command-line examples.

Could you please apply the same modification to docs/content to ensure consistency?

Thanks in advance!

@hugomatheuss
Copy link
Author

Hi @tigitz, thanks for the feedback!
I understood about what you say, so I applied the changes as you said.
Is here in this new PR #54
Thanks again for the guidance and the opportunity to contribute to such a great project!
Bests regards!

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