Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

just-the-docs を 0.7.0 までアップデートする #1399

Merged
merged 5 commits into from
Jan 22, 2024

Conversation

muryoimpl
Copy link
Contributor

@muryoimpl muryoimpl commented Nov 23, 2023

just-the-docs を 0.7.0 まで上げました。

また、環境構築時にliquid のバージョンが古いものが導入されて環境構築に失敗したため、Gemfile.lock を導入して各種Gemのバージョンを固定しています。
just-the-docs 本家に dependabot が導入されたので、just-the-docs を上げていけば周辺Gemも一緒に上がるようになります。meetupで独自に導入しているものは自分たちで上げないといけないけど、これはまた別で対処します。

[注意]
Gemfile.lock を導入しましたが、Platform のところは MacOS sonoma の darwin が記載されています。sonoma でない環境で bundle install すると差分が出てしまう可能性はありますが、それはその都度対応します。

Copy link

netlify bot commented Nov 23, 2023

Deploy Preview for meetup-kzrb-org ready!

Name Link
🔨 Latest commit 09653f2
🔍 Latest deploy log https://app.netlify.com/sites/meetup-kzrb-org/deploys/65a54b44ac32170008cbffb8
😎 Deploy Preview https://deploy-preview-1399--meetup-kzrb-org.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 configuration.

新しい環境を構築しようとした際、Gem のバージョンが安定しないことで
`bundle install` 時に失敗することがあるため、Gemfile.lock をリポジトリ
に導入する。

これまでは最新Gemに追随するためにGemfile.lockをあえて入れずにおく意図
であえて導入していなかったが、環境構築に失敗することによるデメリットと、
just-the-docs 本家が dependabot を導入したことによりjust-the-docs を
更新する限りはGemは更新されていくため、導入に踏み切った。
#1396
@muryoimpl muryoimpl force-pushed the update-just-the-docs-to-0.7.0 branch from f1b71e1 to 09653f2 Compare January 15, 2024 15:12
@izawa
Copy link
Member

izawa commented Jan 22, 2024

macOS Chrome (120.0.6099.234(Official Build) (x86_64)) で一通りのページが問題なく見れました!

Copy link
Member

@kiyohara kiyohara left a comment

Choose a reason for hiding this comment

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

プレビューで確認して問題なさそうでした! approve しますー 💪

@muryoimpl
Copy link
Contributor Author

早速の確認、ありがとうございます!

@muryoimpl muryoimpl merged commit 0ac7d06 into master Jan 22, 2024
4 checks passed
@muryoimpl muryoimpl deleted the update-just-the-docs-to-0.7.0 branch January 22, 2024 03:42
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants