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: prepend spaceship root folder to spaceship precompile path #1244

Merged

Conversation

LuisUrrutia
Copy link
Contributor

Description

When spaceship-prompt precompile libs, it's done by providing the full path to spaceship::precompile, but in the case of prompt_spaceship_setup/spaceship.zsh, that doesn't happen, which leads to issue #1188

This PR prepends the $SPACESHIP_ROOT var to the arguments of that line to provide the full path of the file.

@netlify
Copy link

netlify bot commented Sep 29, 2022

Deploy Preview for spaceship-prompt ready!

Name Link
🔨 Latest commit 9393dce
🔍 Latest deploy log https://app.netlify.com/sites/spaceship-prompt/deploys/63358b632789990009a3ec41
😎 Deploy Preview https://deploy-preview-1244--spaceship-prompt.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@denysdovhan denysdovhan merged commit 6c4623f into spaceship-prompt:master Sep 29, 2022
@denysdovhan
Copy link
Member

Thanks for contributing!

github-actions bot pushed a commit that referenced this pull request Sep 30, 2022
## [4.5.1](v4.5.0...v4.5.1) (2022-09-30)

### Bug Fixes

* **kubectl:** Avoids checking kubectl server version if no context is loaded, stopping potential hangs ([#1246](#1246)) ([44819b0](44819b0))
* prepend spaceship root folder to spaceship precompile path ([#1244](#1244)) ([6c4623f](6c4623f))
* upsearch was exiting when the first file is not found ([#1239](#1239)) ([dcfd690](dcfd690))
@denysdovhan
Copy link
Member

🎉 This PR is included in version 4.5.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@denysdovhan denysdovhan added the released Released issues and PRs label Sep 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released Released issues and PRs
Development

Successfully merging this pull request may close these issues.

None yet

2 participants