Skip to content

fix: incorrect script paths in stop.sh and start.sh#203

Merged
kevinaimonster merged 1 commit intomindverse:masterfrom
mdqst:patch-1
Apr 11, 2025
Merged

fix: incorrect script paths in stop.sh and start.sh#203
kevinaimonster merged 1 commit intomindverse:masterfrom
mdqst:patch-1

Conversation

@mdqst
Copy link
Copy Markdown
Contributor

@mdqst mdqst commented Apr 10, 2025

i noticed an issue with the paths to the stop.sh and start.sh scripts. they were relative paths, which could break if the script is run from a different directory.

i've updated the paths to use the ${SCRIPT_DIR} variable, which ensures the correct absolute path is used, regardless of the current working directory.

this should prevent any errors related to incorrect paths when invoking the scripts.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 10, 2025

CLA assistant check
All committers have signed the CLA.

Copy link
Copy Markdown
Contributor

@kevinaimonster kevinaimonster left a comment

Choose a reason for hiding this comment

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

good job!

@kevinaimonster kevinaimonster merged commit 98594ee into mindverse:master Apr 11, 2025
1 check passed
@ScarletttMoon
Copy link
Copy Markdown
Collaborator

Hi @mdqst 👋,

Thank you so much for your contribution to this PR! Your work is really appreciated. If you haven’t already, feel free to join our Discord community here: Discord Invite Link — it's a great place to connect with our team and other contributors, share ideas, and stay up to date with the project! You can find me as @scarlettt_moon there!

Looking forward to connecting! 😊

Heterohabilis pushed a commit to Heterohabilis/Second-Me that referenced this pull request May 29, 2025
EOMZON pushed a commit to EOMZON/Second-Me that referenced this pull request Feb 1, 2026
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.

5 participants