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

Add rvm-windows to installation doc #3250

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

magynhard
Copy link

Add the new RVM 4 Windows to the list of Ruby version managers at the installation documentation.

Included languages are bg de en es fr id it ja ko pl pt ru tr zh_cn

@magynhard magynhard requested review from a team as code owners May 20, 2024 00:07
@vtamara
Copy link
Contributor

vtamara commented May 20, 2024

Except for trailing white space. Spanish is good.

magynhard and others added 4 commits May 20, 2024 21:12
…umentation' into feature-add_rvm_4_windows_to_documentation

# Conflicts:
#	es/documentation/installation/index.md
@magynhard magynhard requested a review from marocchino May 20, 2024 20:31
@magynhard
Copy link
Author

Except for trailing white space. Spanish is good.

@vtamara trailing whitespaces removed

Comment on lines +341 to +342
### RVM 4 Windows
{: #rvm-windows}
Copy link
Member

Choose a reason for hiding this comment

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

RVM 4 Windows should be added to TOC between "RVM" and "uru."

Copy link
Author

Choose a reason for hiding this comment

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

My idea was to make it analogous to "rbenv for windows", which also does not have its own entry in TOC, but follows "rbenv". So does "RVM 4 Windows" follow "RVM".

Copy link
Member

Choose a reason for hiding this comment

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

Oh, I understand.

@magynhard magynhard requested a review from yous May 21, 2024 14:36
@magynhard
Copy link
Author

Hey guys,

for me it looks like, that many languages are even not maintained anymore, as no more reviews happend within one month.

So, would you suggest me, to make a new PR only with Korean and English? Or is there a chance to get this PR ever merged without all languages reviewed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants