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

Webテキストに合わせて 7.0 を Codespaces に対応させました #85

Merged
merged 8 commits into from
Feb 9, 2024

Conversation

rakuda-san-desu
Copy link
Contributor

@rakuda-san-desu rakuda-san-desu commented Jan 17, 2024

やったこと

  • 7.0を Codespaces で動作確認 / 作成したコードに更新しました
  • READMEの内容を調整しました

やること

  • 当PRからREADMEの手順で動かせるか確認
  • 確認後レビュー依頼

@rakuda-san-desu rakuda-san-desu changed the title Webテキストに合わせて 7.0 を Codespaces に対応させました wip: Webテキストに合わせて 7.0 を Codespaces に対応させました Jan 17, 2024
@rakuda-san-desu rakuda-san-desu changed the title wip: Webテキストに合わせて 7.0 を Codespaces に対応させました Webテキストに合わせて 7.0 を Codespaces に対応させました Jan 29, 2024
@Yuppymam Yuppymam self-requested a review February 3, 2024 07:53
README.md Outdated
$ git push -u origin main
```

1. 手順1で作成したリポジトリ(ここではsample_ch11)で、「Code」から「Codespaces」タブに移動し、「Create codespace on main」をクリックすると環境構築がスタートします。しばらく待つとシンプルブラウザに「ActiveRecord::PendingMigrationError」が表示されるので、少し下の「Run pending migrations」ボタンをクリックするか、ターミナルを開いて以下のコマンドを実行するとデータベースが移行されます。
Copy link
Contributor

Choose a reason for hiding this comment

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

手順1で作成したリポジトリ

リポジトリ作成を「手順2」に変更いただいたので、追従お願いします!😊

Copy link
Contributor Author

Choose a reason for hiding this comment

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

573a742 で対応しました🛠

```
$ git clone https://github.com/yasslab/sample_apps.git
$ cd 7_0/ch11
```

2. 本番環境でのみ必要なgemはスキップして`bundle install`し、必要なフレームワークをインストールします。この時**conflictが起こった場合は、全て「n(上書きしない)」を選択**してください
2. 本番環境でのみ必要なgemはスキップして`bundle install`します
Copy link
Contributor

Choose a reason for hiding this comment

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

bundle _2.4.12_ installだけだと動かなかったのでbundle _2.4.12_ updateの記載、もしくは「1.3.1Bundler」の最後に説明があるので、セクションへのリンクがあると嬉しいかもです!✨

Copy link
Contributor Author

Choose a reason for hiding this comment

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

「Codespaces 以外の環境で動かす場合」を、参考情報としました🙏
内容もシンプルに、バージョンなしに戻しています。

掲載の有無も含めて、どの程度詳しくするかは別issueにてまとめて考えようと思います🙇‍♀️

Copy link
Contributor

@Yuppymam Yuppymam left a comment

Choose a reason for hiding this comment

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

ご対応ありがとうございました🌷

@rakuda-san-desu
Copy link
Contributor Author

レビューありがとうございます!
マージします🚀

@rakuda-san-desu rakuda-san-desu merged commit cc0231e into main Feb 9, 2024
3 checks passed
@rakuda-san-desu rakuda-san-desu deleted the codespaces_adjustment branch February 9, 2024 01:19
@rakuda-san-desu rakuda-san-desu restored the codespaces_adjustment branch February 19, 2024 07:59
@rakuda-san-desu rakuda-san-desu deleted the codespaces_adjustment branch February 19, 2024 07:59
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.

None yet

2 participants