-
-
Notifications
You must be signed in to change notification settings - Fork 965
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
feat: add support for R lang #1371
base: master
Are you sure you want to change the base?
feat: add support for R lang #1371
Conversation
✅ Deploy Preview for spaceship-prompt ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about using rlang
for section name? What do you think?
docs/sections/R.md
Outdated
@@ -0,0 +1,25 @@ | |||
# R (`R lang`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's use a convention similar to V lang. We use vlang
key in code to avoid confusion. Let's do the same for R lang. Please use rlang
everywhere
# R (`R lang`) | |
# R (`rlang`) |
docs/sections/R.md
Outdated
!!! info | ||
[**R**](https://r-project.org) is a free software environment for statistical computing and graphics. | ||
|
||
The `R` section displays the R version. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The `R` section displays the R version. | |
The `rlang` section displays the R version. |
docs/sections/R.md
Outdated
|
||
| Variable | Default | Meaning | | ||
| :-------------------- | :--------------------------------: | ------------------------------------------ | | ||
| `SPACESHIP_R_SHOW` | `true` | Show section | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
R
-> RLANG
| `SPACESHIP_R_SHOW` | `true` | Show section | | |
| `SPACESHIP_RLANG_SHOW` | `true` | Show section | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, @denysdovhan. I'll do that.
In the meantime, can you have a look at other PR #1367?
Hey @denysdovhan, I have made the changes as you have suggested. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Small mistake in sections code.
- Please add rlang to
mkdocs.yml
table of content. - Add rlang to
internal.json
sections/rlang.zsh
Outdated
[[ $? -ne 0 || -z $R_version ]] && return | ||
|
||
# Verbose output | ||
[[ $SPACESHIP_R_VERBOSE == false ]] && R_version=$(echo $R_version | cut -d ' ' -f 1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[[ $SPACESHIP_R_VERBOSE == false ]] && R_version=$(echo $R_version | cut -d ' ' -f 1) | |
[[ $SPACESHIP_RLANG_VERBOSE == false ]] && R_version=$(echo $R_version | cut -d ' ' -f 1) |
Description
This PR adds support for R lang.
Fixes #1224.
Screenshot