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

설치 안내문 업데이트 #51

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DFDRDODM95
Copy link

이전에 이슈를 등록했었는데
풀 리퀘스트도 해보고 싶어서 요청드렸습니다.

새 아이디를 하나 만들어서 설치 과정을 다시 진행하고
녹화한 영상을 보면서 작업 순서대로 적어봤습니다.

Copy link
Owner

@johngrib johngrib left a comment

Choose a reason for hiding this comment

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

좋습니다. 훌륭한 PR입니다.

  • PR 타이틀을 수정해 주세요. 설치 안내문 업데이트 정도면 좋을 것 같습니다.
  • 설명이 길어지니 install.sh 파일을 따로 제공하는 것도 괜찮을 것 같군요.

```

skeleton을 fork한 원격저장소를 clone하고
해당 폴더로 이동합시다.
Copy link
Owner

Choose a reason for hiding this comment

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

'폴더'보다 '디렉토리'를 사용해 주세요.

Suggested change
해당 폴더로 이동합시다.
해당 디렉토리로 이동합시다.

Comment on lines +35 to +40
```bash
$ git clone https://github.com/rbenv/ruby-build.git "$(rbenv root)"/plugins/ruby-build
$ sudo apt-get install libffi-dev
$ sudo apt-get install libyaml-dev
$ rbenv install 3.2.3
```
Copy link
Owner

@johngrib johngrib Mar 3, 2024

Choose a reason for hiding this comment

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

macOS 사용자(저)를 위한 방법으로 brew install rbenv를 사용하는 방법을 추가해주면 좋겠습니다.
만약 macOS 컴퓨터 사용이 곤란해서 관련 내용을 추가하기 어렵다면 말씀해주세요. 제가 추가하면 됩니다.

export PATH="$HOME/.gems/bin:$PATH"
```

skeleton을 fork한 원격저장소를 clone하고
Copy link
Owner

Choose a reason for hiding this comment

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

clone을 언급하게 되었으므로 git clone 명령도 여기에서 소개해주는 것이 바람직할 것 같습니다.

Copy link
Author

Choose a reason for hiding this comment

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

업데이트를 한 후 리뷰 요청 드리겠습니다!
install.sh를 준비하고 싶어서 한참 뒤에 요청이 업데이트 될 거 같습니다.
자세하게 리뷰해주셔서 감사합니다.

@johngrib johngrib self-assigned this Mar 3, 2024
@DFDRDODM95 DFDRDODM95 changed the title README.md 페이지를 수정했습니다. ruby 버전과 설치 방식을 바꿔 썼어요. 설치 안내문 업데이트 Mar 3, 2024
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