Skip to content

fix: directory in quickstart command#56

Merged
srijanpatel merged 1 commit intoPySpur-Dev:mainfrom
NiloCK:patch-1
Dec 19, 2024
Merged

fix: directory in quickstart command#56
srijanpatel merged 1 commit intoPySpur-Dev:mainfrom
NiloCK:patch-1

Conversation

@NiloCK
Copy link
Copy Markdown
Contributor

@NiloCK NiloCK commented Dec 19, 2024

-bash: cd: pyspur: No such file or directory

You know what they say about first impressions :)

Looks very cool.


Important

Fix directory name in README.md quickstart command to match cloned repository name.

  • Documentation:
    • Fix directory name in README.md quickstart command from cd pyspur to cd PySpur to match the cloned repository name.

This description was created by Ellipsis for ac0cb94. It will automatically update as commits are pushed.

Copy link
Copy Markdown
Contributor

@ellipsis-dev ellipsis-dev Bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to ac0cb94 in 7 seconds

More details
  • Looked at 13 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. README.md:47
  • Draft comment:
    The directory name should match the repository name, which is case-sensitive. This change is correct.
  • Reason this comment was not posted:
    Confidence changes required: 0%
    The PR correctly updates the directory name in the README to match the actual repository name, which is case-sensitive.

Workflow ID: wflow_DQ0nG6pYstq19UNI


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@srijanpatel srijanpatel merged commit ada09e4 into PySpur-Dev:main Dec 19, 2024
@srijanpatel
Copy link
Copy Markdown
Collaborator

Thank you @NiloCK !

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