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

Remove $ from README shell scripts #1638

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

Conversation

conbon
Copy link

@conbon conbon commented Nov 7, 2023

Remove $ from README shell scripts.
This hinders copying the instructions using Github UI and slows down users trying it out for the first time.

@conbon conbon requested review from bkizer-tenable and a team as code owners November 7, 2023 08:50
Copy link

sonarcloud bot commented Nov 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@codecov-commenter
Copy link

Codecov Report

Merging #1638 (35ba928) into master (2029321) will not change coverage.
The diff coverage is n/a.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1638   +/-   ##
=======================================
  Coverage   78.49%   78.49%           
=======================================
  Files         280      280           
  Lines        7906     7906           
=======================================
  Hits         6206     6206           
  Misses       1329     1329           
  Partials      371      371           

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.

None yet

2 participants